[JSC] Fix WebKitAddition directory traversal in offlineasm
Created attachment 437219 [details] Patch
Comment on attachment 437219 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=437219&action=review r=me > Source/JavaScriptCore/offlineasm/asm.rb:338 > + opts.banner = "Usage: asm.rb asmFile offsetsFile outputFileName [--assembler=<ASM> --use-deployment-location]" Should this be "[--assembler=<ASM>] [--use-deployment-location]" instead since the 2 options are independent? > Source/JavaScriptCore/offlineasm/generate_offset_extractor.rb:52 > + # This option is currently only used to specify the masm assembler This comment is not relevant. I think it's just copied from above. Please remove. > Source/JavaScriptCore/offlineasm/generate_settings_extractor.rb:49 > + # This option is currently only used to specify the masm assembler Ditto. Please remove.
Comment on attachment 437219 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=437219&action=review Thanks! >> Source/JavaScriptCore/offlineasm/asm.rb:338 >> + opts.banner = "Usage: asm.rb asmFile offsetsFile outputFileName [--assembler=<ASM> --use-deployment-location]" > > Should this be "[--assembler=<ASM>] [--use-deployment-location]" instead since the 2 options are independent? Changed. >> Source/JavaScriptCore/offlineasm/generate_offset_extractor.rb:52 >> + # This option is currently only used to specify the masm assembler > > This comment is not relevant. I think it's just copied from above. Please remove. Removed. >> Source/JavaScriptCore/offlineasm/generate_settings_extractor.rb:49 >> + # This option is currently only used to specify the masm assembler > > Ditto. Please remove. Removed.
Comment on attachment 437219 [details] Patch You missed one spot in parser.rb: search for "/usr/local/include/WebKitAdditions/" and you'll see a seconds place the path is chosen in `def parseIncludes(final, comment)`.
Created attachment 437225 [details] Patch
Comment on attachment 437225 [details] Patch r=me
<rdar://problem/82701787>
OK, compilation finished.
Committed r281972 (241279@main): <https://commits.webkit.org/241279@main>