- Notifications
You must be signed in to change notification settings - Fork3.1k
Make scalac's argument-file processing more like javac's in handling spaces and line breaks#10319
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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
Uh oh!
There was an error while loading.Please reload this page.
Conversation
michelou commentedFeb 19, 2023 • edited
Loading Uh oh!
There was an error while loading.Please reload this page.
edited
Uh oh!
There was an error while loading.Please reload this page.
@som-snytt Your changes work for me, thanks. I closed PR#10316 in favor of your PR after testing your changes with several Scala 2 code examples. I also patched the 2.13.10 installation (i.e. |
som-snytt commentedFeb 20, 2023
@michelou Thanks for trying out the fix. I will probably not further tweak to be bug-compatible with Java. To summarize, the change in 2.13.10 arrived in 2.13.9 but deemed too late in the cycle to refine, so it was reverted and deferred, but then did not receive further scrutiny; I see it got the "release notes" label, but only the I was hoping to use the same tokenizer for I'd forgotten this bug mentioned in my PR, which makes it sound like JDK has quoting bugs in both places: The context is complicated by the reverts, but the original PRs were: |
SethTisue left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others.Learn more.
thanks@michelou for bringing this to our attention, and thanks@som-snytt for untangling the history
SethTisue commentedMar 13, 2023 • edited
Loading Uh oh!
There was an error while loading.Please reload this page.
edited
Uh oh!
There was an error while loading.Please reload this page.
@som-snytt I improved the title and PR description a bit. not sure whether to put additional effort into it; on the one hand it's an incompatible change, so people need to know, but otoh I doubt the feature is widely used |
som-snytt commentedMar 13, 2023
@SethTisue yes, I think is really a reversion. If I could go back in time and live my life over again, this is what I would do differently. I only work on features that nobody uses, so it should be safe. |
michelou commentedMar 22, 2023
@SethTisue Just one reminder to make sure this fix will go into release 2.13.11 :Kotlin andScala 3 do behave the same asJava in respect to the handling of argument files passed on the command line. PS. I make use of argument files in most build scripts (bash, batch, Makefile) available from my Github repositories (e.g. |
Uh oh!
There was an error while loading.Please reload this page.
Fixesscala/bug#12732; the ticket has details on the behavior differences