RFR: 8014003: New build does not handle symlinks in workspace path

Erik Joelsson erik.joelsson at oracle.com
Thu May 30 09:22:49 UTC 2013


The bash builtin version of pwd seems to behave more uniformly. Changed 
to use this version of pwd and also explicitly added either -L or -P to 
all invocations of it to make sure the expected result was generated.

http://cr.openjdk.java.net/~erikj/8014003/webrev.root.02/

/Erik

On 2013-05-29 13:44, Erik Joelsson wrote:
> Unfortunately the /bin/pwd on fedora 9 doesn't follow this. I will try 
> to figure out something else.
>
> /Erik
>
> On 2013-05-29 05:38, Mike Duigou wrote:
>> http://pubs.opengroup.org/onlinepubs/009695399/utilities/pwd.html
>>
>> Yep, -L is part of IEEE Std 1003.1-2001 standard which basically 
>> everything in use today supports. If only the same could be said 
>> about C99.
>>
>> Mike
>>
>>
>> On May 28 2013, at 19:40 , Tim Bell wrote:
>>
>>> Erik:
>>>
>>>> Due to a difference in the default output of the pwd command on mac 
>>>> vs linux and solaris, configure wouldn't allow the source root to 
>>>> be a symlink on mac. This patch fixes this by adding -L to the pwd 
>>>> command, forcing it to show the logical directory rather than the 
>>>> symlink free one.
>>>>
>>>> http://cr.openjdk.java.net/~erikj/8014003/webrev.root.01/
>>> Looks good to me.
>>>
>>> I checked pwd on Solaris, Linux, Cygwin on Windows, and Mac.  They 
>>> all show -L as an option.  Unfortunately, I don't have the means at 
>>> this time to check other platforms such as (non-Mac) BSD, AIX, 
>>> HP-UX, Unicos, etc.
>>>
>>>
>>> Tim
>>>
>>>



More information about the build-dev mailing list