Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

As of Haxe 4.3.5 our project compile never finishes #11820

Open
singpolyma opened this issue Nov 11, 2024 · 6 comments
Open

As of Haxe 4.3.5 our project compile never finishes #11820

singpolyma opened this issue Nov 11, 2024 · 6 comments
Assignees
Milestone

Comments

@singpolyma
Copy link
Contributor

See https://github.com/snikket-im/snikket-sdk on 4.3.2 or 4.3.4 running haxe browserjs.hxml for example finished in a second or two. As of 4.3.5 and 4.3.6 instead it never finishes. Running with --verbose shows and end like this:

Typing haxe.format.JsonPrinter.ipad
Typing haxe.format.JsonPrinter.quote
Parsed /tmp/haxe-4.3.5-linux64/haxe_20240718142605_bd79571/std/Date.hx
Typing haxe.format.JsonPrinter.classString
Parsed /tmp/haxe-4.3.5-linux64/haxe_20240718142605_bd79571/std/haxe/ds/IntMap.hx
Parsed /tmp/haxe-4.3.5-linux64/haxe_20240718142605_bd79571/std/haxe/ds/ObjectMap.hx

then it just hangs there.

@singpolyma
Copy link
Contributor Author

This may be related to #11811 though it seems theirs eventually finishes (our never does, after over a half hour) and the log smells different. But the versions affected are certainly the same.

@singpolyma
Copy link
Contributor Author

Bisect shows that the first bad commit is fbec81a

@singpolyma
Copy link
Contributor Author

Confirmed that if I checkout 4.3.6 and then revert fbec81a that the result can build out code just fine and quickly.

@singpolyma
Copy link
Contributor Author

Just deleting the line let e2 = loop e2 in is enough to make the compile finish

@singpolyma
Copy link
Contributor Author

singpolyma commented Nov 12, 2024

Could you try if it works when you move that line below the let m = Abstract.get_underlying_type a pl in line?

yes moving it there also fixes it

Simn added a commit that referenced this issue Nov 12, 2024
@Simn Simn added this to the 4.3 Hotfix milestone Nov 12, 2024
@Simn
Copy link
Member

Simn commented Nov 12, 2024

@FrancisBourre Could you check if this fixed your problem too?

Simn added a commit that referenced this issue Nov 18, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants