This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [committed] Fix up run_all.sh
- From: Mike Stump <mikestump at comcast dot net>
- To: Jakub Jelinek <jakub at redhat dot com>
- Cc: gcc-patches at gcc dot gnu dot org
- Date: Wed, 17 Sep 2014 12:18:46 -0700
- Subject: Re: [committed] Fix up run_all.sh
- Authentication-results: sourceware.org; auth=none
- References: <20140917182527 dot GU17454 at tucnak dot redhat dot com> <73AAABAA-BE48-4B44-9B25-EE1C52821A46 at comcast dot net> <20140917191620 dot GV17454 at tucnak dot redhat dot com>
On Sep 17, 2014, at 12:16 PM, Jakub Jelinek <jakub@redhat.com> wrote:
> On Wed, Sep 17, 2014 at 12:13:51PM -0700, Mike Stump wrote:
>> On Sep 17, 2014, at 11:25 AM, Jakub Jelinek <jakub@redhat.com> wrote:
>>> - if mkdir $GCC_RUNTEST_PARALLELIZE_DIR/$par_count; then
>>> + if mkdir $GCC_RUNTEST_PARALLELIZE_DIR/$par_count 2>/dev/null; then
>>
>> So, I can’t help but think we should just do a mkdir -p for this and be done with it.
>
> mkdir -p does not fail if the directory exists, which is exactly what this
> code relies on for the synchronization.
Oh, yeah, right. :-) Never mind.