[arch-releng] [PATCH] Removes incorrect quotes in automated_script
Gerardo Exequiel Pozzi
vmlinuz386 at yahoo.com.ar
Tue Apr 26 19:08:17 EDT 2011
On 04/26/2011 06:01 PM, Dieter Plaetinck wrote:
> On Tue, 26 Apr 2011 10:34:54 -0300
> Gerardo Exequiel Pozzi<vmlinuz386 at yahoo.com.ar> wrote:
>
>> On 04/26/2011 09:08 AM, Charles Vejnar wrote:
>>> ---
>>> .../overlay/etc/rc.d/functions.d/automated_script | 2 +-
>>> 1 files changed, 1 insertions(+), 1 deletions(-)
>>>
>>> diff --git a/configs/syslinux-iso/overlay/etc/rc.d/functions.d/automated_script b/configs/syslinux-iso/overlay/etc/rc.d/functions.d/automated_script
>>> index 076a49c..7a35acb 100644
>>> --- a/configs/syslinux-iso/overlay/etc/rc.d/functions.d/automated_script
>>> +++ b/configs/syslinux-iso/overlay/etc/rc.d/functions.d/automated_script
>>> @@ -3,7 +3,7 @@ automated_script ()
>>> script="$(kernel_cmdline script)"
>>> if [[ -n "$script" ]]; then
>>> stat_busy "Configuring $script"
>>> - if [[ "$script" =~ "^http://" || "$script" =~ "^ftp://" ]]; then
>>> + if [[ "$script" =~ ^http:// || "$script" =~ ^ftp:// ]]; then
>>> wget "$script" -q -O /tmp/startup_script>/dev/null
>>> rt=$?
>>> else
>> pushed to master. Thanks.
>>
>
> What?
> First you commit the useless quotes because they are "cosmetic".
Cosmetic in "$script" but regex.
> http://mailman.archlinux.org/pipermail/arch-releng/2011-April/001671.html
> Then you remove them again in a new commit.
>
> git history pollution...
I hope this disaster will not happen again...
> Dieter
>
--
Gerardo Exequiel Pozzi
\cos^2\alpha + \sin^2\alpha = 1
More information about the arch-releng
mailing list