[Top][All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [patch] install with a missing strip tool

From: Bernhard Voelker
Subject: Re: [patch] install with a missing strip tool
Date: Fri, 22 Feb 2013 14:18:44 +0100
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:17.0) Gecko/20130105 Thunderbird/17.0.2

On 02/21/2013 08:40 PM, Ondrej Oprala wrote:
> so let's do your approach ;)

Hi Ondrej,

thank you.

In your patch, you missed the case when fork failed:
the target would yet be left behind.

Therefore, and to have slightly smaller code, I inverted the
logic in strip(), and used a bool return value instead of int.

The inclusion of unistd.h was unnecessary BTW: the above are already included via system.h

Since this was reported in Redhat's Bugzilla, and it changes
the behavior in the case of such an error for the user,
I've also added a NEWS entry.

Finally, I've removed the error suppression in the test,
as it may help to diagnose problems in future.
I also changed the target file name to avoid interference
with the first test (paranoia, I know... ;-)

Is this okay for you?

Have a nice day,

Attachment: install-strip-cleanup.patch
Description: Text Data

reply via email to

[Prev in Thread] Current Thread [Next in Thread]