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

-seq -timeout 15m still hitting timeout #6

Closed
PXshadow opened this issue Oct 5, 2022 · 3 comments
Closed

-seq -timeout 15m still hitting timeout #6

PXshadow opened this issue Oct 5, 2022 · 3 comments

Comments

@PXshadow
Copy link
Collaborator

PXshadow commented Oct 5, 2022

go2hx's action runner is still hitting timeout when calling go2hxdoc:

./go2hxdoc -dir `npx haxelib libpath go2hx`stdgo -prefix npx -interp -jvm -hl -stats -force -seq -timeout 15m
@elliott5
Copy link
Collaborator

What has perplexed me about this situation is that the -interp hxml is much simpler than the doc-generation hxml, yet the doc-generation hxml is running and producing xml output that is being further processed.

The working doc-generation hxml is being run by invoking haxe without any -prefix:

cmd := exec.Command("haxe", comp.HxmlDoc)

While the failing tests are being run with the -prefix npx flag (a " " is added to the end of CmdPrefix, if present, before this code executes):

cmd := exec.CommandContext(ctx, CmdPrefix+"haxe", hxml)

@PXshadow - please could we try it again without the -prefix npx flag?

I would expect the -interp tests to run, although the -hl and -jvm tests may fail, since their file paths may need to be corrected using filepath.Abs() as per: 56a38b1

@PXshadow
Copy link
Collaborator Author

What has perplexed me about this situation is that the -interp hxml is much simpler than the doc-generation hxml, yet the doc-generation hxml is running and producing xml output that is being further processed.

The working doc-generation hxml is being run by invoking haxe without any -prefix:

cmd := exec.Command("haxe", comp.HxmlDoc)

While the failing tests are being run with the -prefix npx flag (a " " is added to the end of CmdPrefix, if present, before this code executes):

cmd := exec.CommandContext(ctx, CmdPrefix+"haxe", hxml)

@PXshadow - please could we try it again without the -prefix npx flag?

I would expect the -interp tests to run, although the -hl and -jvm tests may fail, since their file paths may need to be corrected using filepath.Abs() as per: 56a38b1

It's running now without the -prefix npx The issue I think is still occurring @elliott5 feel free to look at the runner, it's now passing all of the current tests.

@PXshadow
Copy link
Collaborator Author

No issue anymore with the runner.

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