<div dir="ltr"><div>Hi Pete,</div><div><br></div><div>Looks like both ld and gold don't think that @ is a valid character in a token, so I think we shouldn't, too.</div><div><pre style="color:rgb(0,0,0)">$ ld -script <(echo 'SEARCH_DIR( /Volumes/Data/apple-lld-stage1@2/ )')
ld:/dev/fd/63:1: ignoring invalid character `@' in expression
ld:/dev/fd/63:1: syntax error</pre><pre><font face="arial, sans-serif"><span style="white-space:normal">Do you think you can fix the tests by using relative path instead of absolute path?</span></font></pre></div></div><div class="gmail_extra"><br><div class="gmail_quote">On Fri, Jan 22, 2016 at 12:48 PM, Pete Cooper <span dir="ltr"><<a href="mailto:peter_cooper@apple.com" target="_blank">peter_cooper@apple.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi Rui, Rafael<br>
<br>
I have a jenkins bot building lld.  Due to a plugin called ‘shared workspaces’ its common to have @’s in paths.<br>
<br>
This bot is then failing on 3 ELF tests (ELF/as-needed.s, ELF/linkerscript.s, ELF/linkerscript2.s)<br>
<br>
I’ve diagnosed the issue on linkerscript.s and its down to @ not being in the set of unquoted tokens in LinkerScript::tokenize.<br>
<br>
I’d like to propose a patch, but first I’d like to double check what the preferred solution is.  I think its either:<br>
- Add @ to the allowable characters in a token, or<br>
- Change the test to quote the path<br>
<br>
Which would you prefer?<br>
<br>
Thanks,<br>
Pete<br>
<br>
PS, the linkerscipt.s script contains this entry which is the cause of the failure: "SEARCH_DIR( /Volumes/Data/apple-lld-stage1@2/build/tools/lld/test/ELF/Output/linkerscript2.s.tmp.dir )"</blockquote></div><br></div>