Skip to content

Conversation

@dellis1972
Copy link
Contributor

@dellis1972 dellis1972 commented May 22, 2018

Context https://devdiv.visualstudio.com/DevDiv/_workitems/edit/599161

We currently only use a response file for .java files.
Looking at the docs there is no reason why we can't use
them for all of the arguments. This should help with
problems where windows has a max command line length.

@dellis1972 dellis1972 added the do-not-merge PR should not be merged. label May 22, 2018
@jonpryor
Copy link
Contributor

As pointed out on our call, while using a javac response file may overall be a Good Idea™, it will not fix the System.IO.PathTooLongException, as that results from attempting to add a file in a "too long" path into classes.zip after running javac.

javac ran successfully. As such, it's not part of the problem.

@dellis1972 dellis1972 removed the do-not-merge PR should not be merged. label Jul 2, 2018
@dellis1972 dellis1972 force-pushed the javacresponse branch 4 times, most recently from 8723c90 to 161239f Compare September 10, 2018 15:17
@dellis1972
Copy link
Contributor Author

build

@dellis1972
Copy link
Contributor Author

build

@dellis1972 dellis1972 force-pushed the javacresponse branch 2 times, most recently from 52c5f93 to 828b9b2 Compare September 12, 2018 12:41
@dellis1972
Copy link
Contributor Author

@jonp the failing test is

RemotingServicesTest.Xamarin.Android.Bcl_Tests, MonoTests.System.Runtime.Remoting.RemotingServicesTest.Bug46473 / Debug

So I think this is good to go!!

@dellis1972
Copy link
Contributor Author

ok this is good. Test is un related .ConditionalWeakTableTest.OldGenStress / Debug

@dellis1972 dellis1972 force-pushed the javacresponse branch 2 times, most recently from d3259da to 36ea114 Compare September 18, 2018 16:00
@dellis1972
Copy link
Contributor Author

build

1 similar comment
@dellis1972
Copy link
Contributor Author

build

@jonpryor
Copy link
Contributor

macOS PR Build log shows the problem: too many quotes in <Javac/>:

    Javac Task (TaskId:187)
...
    -d ""obj/Test Config/android/bin/classes"" (TaskId:187)
    -bootclasspath "/Users/builder/android-toolchain/sdk/platforms/android-28/android.jar" (TaskId:187)
    -classpath "/Users/builder/jenkins/workspace/xamarin-android-pr-builder/xamarin-android/bin/Debug/lib/xamarin.android/xbuild-frameworks/MonoAndroid/v9.0/mono.android.jar:/Users/builder/jenkins/workspace/xamarin-android-pr-builder/xamarin-andro
id/bin/Debug/lib/xamarin.android/xbuild/Xamarin/Android/android-support-multidex.jar" (TaskId:187)
    -encoding UTF8 (TaskId:187)
    /Library/Java/JavaVirtualMachines/jdk1.8.0_91.jdk/Contents/Home/bin/javac -J-Dfile.encoding=UTF8 "@/var/folders/19/hjm21bn17913_0ck7nh3thdr0000gn/T/tmp546e6a6.tmp" -target 1.7 -source 1.7  (TaskId:187)
    javac: invalid flag: obj/Test (TaskId:187)
    Usage: javac <options> <source files> (TaskId:187)
    use -help for a list of possible options (TaskId:187)

@dellis1972
Copy link
Contributor Author

still green......

@dellis1972 dellis1972 force-pushed the javacresponse branch 2 times, most recently from 8083f3d to 73f97d9 Compare January 28, 2019 11:55
@dellis1972 dellis1972 force-pushed the javacresponse branch 3 times, most recently from 3237a89 to 793c9f6 Compare July 12, 2019 09:37
@dellis1972 dellis1972 force-pushed the javacresponse branch 5 times, most recently from 0a5cf0a to fe2543a Compare August 28, 2019 15:47
Context https://devdiv.visualstudio.com/DevDiv/_workitems/edit/599161

We currently only use a response file for .java files.
Looking at the docs there is no reason why we can't use
them for all of the arguments. This should help with
problems where windows has a max command line length.
@jonpryor jonpryor merged commit 0eed072 into dotnet:master Aug 28, 2019
@github-actions github-actions bot locked and limited conversation to collaborators Feb 2, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants