bugGNU GRUB - Bugs: bug #36961, Use of `\t' in sed breaks...

 
 

You are not allowed to post comments on this tracker with your current authentication level.

bug #36961: Use of `\t' in sed breaks grub-mkconfig

Submitted by:  GrĂ©goire Sutre <gsutre>
Submitted on:  Thu 26 Jul 2012 09:57:01 AM UTC  
 
Category: ConfigurationSeverity: Major
Priority: 5 - NormalItem Group: None
Status: FixedPrivacy: Public
Assigned to: NoneOriginator Name: 
Open/Closed: ClosedRelease: 
Release: Bazaar - trunkReproducibility: Every Time
Planned Release: None

Sun 05 Aug 2012 10:04:33 AM UTC, comment #1:

Feel free to introduce a new variable $tab set to a tabulation character and use it wherever we have \t

Vladimir Serbinenko <phcoder>
Project Administrator
Thu 26 Jul 2012 09:57:01 AM UTC, original submission:

Various scripts in util/grub.d use `\t' in sed commands to indent
menu entries.

However, on some systems, sed does not recognize `\t', and the
resulting grub.cfg contains lines starting with 't' instead of
tabulations.

The problem occurs on NetBSD, but I guess that other BSDs are affected (as well as OS X?).

To illustrate the problem in detail:

On Debian GNU/Linux:
$ sed --version
GNU sed version 4.2.1
[...]
$ echo "hello" | sed -e "s/^/\t/"
hello

On NetBSD 6.99.3
$ echo "hello" | sed -e "s/^/\t/"
thello

A simple fix is to replace those `\t' by tabulation characters.

Grégoire Sutre <gsutre>
Project Member

 

No files currently attached

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -unavailable- added by phcoder (Posted a comment)
  • -unavailable- added by gsutre (Submitted the item)
  •  

    Do you think this task is very important?
    If so, you can click here to add your encouragement to it.
    This task has 0 encouragements so far.

    Only logged-in users can vote.

     

    Please enter the title of George Orwell's famous dystopian book (it's a date):

     

     

    Follow 4 latest changes.

    Date Changed By Updated Field Previous Value => Replaced By
    Tue 18 Sep 2012 11:05:04 AM UTCphcoderStatusNone=>Fixed
      Open/ClosedOpen=>Closed
      SummaryUse of `\\t\' in sed breaks grub-mkconfig=>Use of `\t' in sed breaks grub-mkconfig
    Sun 05 Aug 2012 10:04:33 AM UTCphcoderSummaryUse of `\\t\' in sed breaks grub-mkconfig=>Use of `\t' in sed breaks grub-mkconfig

    Back to the top


    Powered by Savane 3.1-cleanup1