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

racket/src/{bc,cs/c}/configure fix arg stripping #3692

Closed
wants to merge 1 commit into from

Conversation

tgbugs
Copy link
Contributor

@tgbugs tgbugs commented Feb 19, 2021

rktio doesn't know how to handle the --collectsdir argument and will
stop the build process if it is passed to racket/src/configure, this
fixes the immediate issue but the underlying issue remains and other
unhandled arguments are likely to break rktio configure as well

rktio doesn't know how to handle the --collectsdir argument and will
stop the build process if it is passed to racket/src/configure, this
fixes the immediate issue but the underlying issue remains and other
unhandled arguments are likely to break rktio configure as well
@mflatt
Copy link
Member

mflatt commented Feb 19, 2021

Thanks for the patch! But I'll adjust the source of the configures scripts and rebuild them. The other argument that could cause problems in the same way is --appsdir.

@mflatt mflatt closed this in 19c3ee4 Feb 19, 2021
maueroats pushed a commit to maueroats/racket that referenced this pull request Jun 17, 2021
Thanks to @tgbugs for the report an initial repair.

Closes racket#3692
maueroats pushed a commit to maueroats/racket that referenced this pull request Jun 17, 2021
Thanks to @tgbugs for the report an initial repair.

Closes racket#3692
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

Successfully merging this pull request may close these issues.

None yet

2 participants