New Ticket     Tickets     Wiki     Browse Source     Timeline     Roadmap     Ticket Reports     Search

Ticket #27330 (closed defect: fixed)

Opened 4 years ago

Last modified 4 years ago

smartmontools @5.40_0: update-smart-drivedb script requires GNU sed

Reported by: takanori@… Owned by: toby@…
Priority: Normal Milestone:
Component: ports Version: 1.9.2
Keywords: gsed Cc:
Port: smartmontools

Description

update-smart-drivedb script doesn't work properly with BSD sed since it contains ERE.

Here is a patch to fix this problem.

Attachments

smartmontools.diff (2.1 KB) - added by takanori@… 4 years ago.
smartmontools.diff
update-smart-drivedb-sed.patch (756 bytes) - added by Christian.Franke@… 4 years ago.
This should work with any sed

Change History

Changed 4 years ago by takanori@…

smartmontools.diff

comment:1 Changed 4 years ago by jmr@…

Couldn't you just use sed -E?

comment:2 Changed 4 years ago by takanori@…

I know TIMTOWTDI. ;-)

In this case, using GNU sed is more reliable than using BSD sed with -E option, I think.

Changed 4 years ago by Christian.Franke@…

This should work with any sed

comment:3 Changed 4 years ago by Christian.Franke@…

Please test update-smart-drivedb-sed.patch with BSD sed if possible.

Then I could fix this upstream (Smartmontools Ticket #126).

comment:4 Changed 4 years ago by takanori@…

  • Status changed from new to closed
  • Resolution set to fixed

Your patch worked perfectly. (tested on Mac OS X 10.6.5/x86_64 and 10.5.8/ppc)

Committed in r73687 - Thanks!

comment:5 Changed 4 years ago by Christian.Franke@…

Thanks for testing. Committed upstream in r3214.

Note: See TracTickets for help on using tickets.