RFR 8210318: idea.sh script doesn't work on Mac
Erik Joelsson
erik.joelsson at oracle.com
Tue Sep 4 17:50:29 UTC 2018
Hello,
$TARGET was just pseudo code. In your case it's $1.tmp.
/Erik
On 2018-09-04 10:34, Maurizio Cimadamore wrote:
> Hi Erik,
> would $TARGET be set by make?
>
> Maurizio
>
>
> On 04/09/18 16:55, Erik Joelsson wrote:
>> Hello,
>>
>> When choosing a temp file in the build, we avoid using /tmp whenever
>> possible. A common pattern is instead to write to $TARGET.tmp and
>> then mv that to $TARGET. Though unlikely to cause an issue,
>> /tmp/replacement is a global location which is potentially risky
>> (file permissions, concurrent execution etc).
>>
>> Otherwise looks good.
>>
>> /Erik
>>
>> On 2018-09-03 05:39, Maurizio Cimadamore wrote:
>>> Hi,
>>> following the latest updates to the idea.sh script, Mac users
>>> reported issues - mostly having to do with usage of 'sed' - more
>>> specifically:
>>>
>>> * sed -i option is not portable - it has different formats in Mac
>>> vs. Linux. This patch does without -i, by moving the replaced file
>>> onto a temporary file, then moving such file on top of the template
>>> file in a subsequent step. This should be more robust.
>>>
>>> * sed doesn't like newlines in replaced text in Mac. I've thus
>>> omitted the newline from the SOURCE template - as that was mostly
>>> cosmetic.
>>>
>>> Thanks for Michael McMahon to report (and figure out how to deal
>>> with) these issues, and to Alan Bateman for testing the patch.
>>>
>>> I also fixed another minor glitch, this time in the langtools-only
>>> template - which was still referring to the old ant file location in
>>> the various run configuration.
>>>
>>> Webrev:
>>>
>>> http://cr.openjdk.java.net/~mcimadamore/8210318/
>>>
>>> Cheers
>>> Maurizio
>>>
>>
>
More information about the compiler-dev
mailing list