ProjectScaffold icon indicating copy to clipboard operation
ProjectScaffold copied to clipboard

Stack overflow exception when generating documentation

Open Vidarls opened this issue 10 years ago • 9 comments

Getting stack overflows today when generating documentation.

Seems it was introduces when you moved to using FSharp.Core as a nuget package.

Clean checkout for 2a87478f5121930f4fec9b9ca1846fc117a5058e seems to work. Clean checkout for 83f0514da822648ad5b1a5efdad3be1e2d21c086 seems to create stack overflow.

Will investigate further.

Vidarls avatar Feb 26 '15 10:02 Vidarls

so

Vidarls avatar Feb 26 '15 10:02 Vidarls

https://github.com/fsprojects/FSharpx.Reflection/issues/1 sems I'm not the only one.

forki avatar Feb 26 '15 10:02 forki

Confirmed, just manually reverted the changes from 83f0514da822648ad5b1a5efdad3be1e2d21c086 and voila, no more stackoverflow exceptions

Vidarls avatar Feb 26 '15 10:02 Vidarls

That is current master minus changes from 83f0514da822648ad5b1a5efdad3be1e2d21c086 is ok,

Vidarls avatar Feb 26 '15 10:02 Vidarls

Will do a pull req

Vidarls avatar Feb 26 '15 10:02 Vidarls

TIL: git revert is very useful :smile:

Vidarls avatar Feb 26 '15 10:02 Vidarls

But removing FSharp.Core package will create other issue.

If FCS creates a StackOverflow on FSharp.Core nuget package then this needs to be investigated. /cc @dsyme @7sharp9 @tpetricek @dungpa

forki avatar Feb 26 '15 10:02 forki

Better with some corner case issues in the master branch than outright stackoverflows? We can go back ot using the nuget package once FCS have sorted the problem

Vidarls avatar Feb 26 '15 10:02 Vidarls

I agree and merged it as a temporary fix.

forki avatar Feb 26 '15 10:02 forki