Bug 493584 - Error in postinstall scriptlet here-document
Summary: Error in postinstall scriptlet here-document
Keywords:
Status: CLOSED NEXTRELEASE
Alias: None
Product: Fedora
Classification: Fedora
Component: memtest86+
Version: rawhide
Hardware: All
OS: Linux
low
medium
Target Milestone: ---
Assignee: Paulo Roma Cavalcanti
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2009-04-02 11:13 UTC by Mads Kiilerich
Modified: 2009-05-19 02:06 UTC (History)
2 users (show)

Fixed In Version: 2.11-9.fc10
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2009-05-12 04:10:04 UTC


Attachments (Terms of Use)

Description Mads Kiilerich 2009-04-02 11:13:58 UTC
Description of problem:

Error in postinstall scriptlet:

  Updating       : memtest86+
/var/tmp/rpm-tmp.73Vbvd: line 9: warning: here-document at line 1 delimited by end-of-file (wanted `EOF')

   ************************************************
   * REMINDER: Please update your /etc/grub.conf  *
   * file and add the memtest86+ boot entry,      *
   * by running /usr/sbin/memtest-setup as root.  *
   ************************************************

   EOF

The EOF marker should not be indented.

AFAIK that kind of messages should not be shown when an RPM is installed.

The unconditional sed'ing in the preuninstall scriptlet also looks dangerous.


Version-Release number of selected component (if applicable):

memtest86+-2.11-5.fc11.i586

Comment 1 Paulo Roma Cavalcanti 2009-04-02 12:56:38 UTC
You are right. 

I will remove the message and let the user
add or remove memtest86+ entries from grub.conf.

Thanks.

Comment 2 Mads Kiilerich 2009-04-02 18:33:13 UTC
I think the best solution would be if it was handled by grubby _exactly_ the same way as kernel installation/upgrades is handled. It looks like new-memtest-pkg is a fork from new-kernel-pkg with minimal changes?

Comment 3 Paulo Roma Cavalcanti 2009-04-02 20:15:34 UTC
Grubby does all the work and only inserts arguments at the end of the line,
and we need "--type=netbsd" exactly as below:

kernel --type=netbsd /memtest86+-2.11

If one rebuilds the memtest86+ .src.rpm using "--with update_grub",
then the entry will be inserted and deleted from grub.conf, but using "sed",
for the argument.

Comment 4 Mads Kiilerich 2009-04-03 00:58:01 UTC
Paulo, it is obvious that you have considered it carefully. As a random user I can only add some opinions and random user feedback:

I am sure the grubby maintainers would like to support your package better (they should!) so that sed'ing isn't needed.

Linux kernels are automatically added to grub.conf when they are installed. If memtest86+ uses the same method as the linux kernel packages then I see no reason why it shouldn't do the same. And when the the package is updated then it should unconditionally update its entry in grub.conf.

But this is chatting and has nothing to do with the issue trached here ;-)

Thanks and roger and out from me.

Comment 5 Paulo Roma Cavalcanti 2009-04-03 12:16:00 UTC
Please, check bug # 493681

If grubby is improved, then it will be easy
to handle memtest86+ needs.

Comment 6 Fedora Update System 2009-04-03 12:20:32 UTC
memtest86+-2.11-6.fc10 has been submitted as an update for Fedora 10.
http://admin.fedoraproject.org/updates/memtest86+-2.11-6.fc10

Comment 7 Fedora Update System 2009-04-03 12:20:47 UTC
memtest86+-2.11-6.fc9 has been submitted as an update for Fedora 9.
http://admin.fedoraproject.org/updates/memtest86+-2.11-6.fc9

Comment 8 Fedora Update System 2009-04-12 11:29:17 UTC
memtest86+-2.11-7.fc10 has been submitted as an update for Fedora 10.
http://admin.fedoraproject.org/updates/memtest86+-2.11-7.fc10

Comment 9 Fedora Update System 2009-04-12 11:29:36 UTC
memtest86+-2.11-7.fc9 has been submitted as an update for Fedora 9.
http://admin.fedoraproject.org/updates/memtest86+-2.11-7.fc9

Comment 10 Fedora Update System 2009-04-13 19:33:02 UTC
memtest86+-2.11-7.fc10 has been pushed to the Fedora 10 testing repository.  If problems still persist, please make note of it in this bug report.
 If you want to test the update, you can install it with 
 su -c 'yum --enablerepo=updates-testing update memtest86+'.  You can provide feedback for this update here: http://admin.fedoraproject.org/updates/F10/FEDORA-2009-3511

Comment 11 Fedora Update System 2009-04-13 19:34:50 UTC
memtest86+-2.11-7.fc9 has been pushed to the Fedora 9 testing repository.  If problems still persist, please make note of it in this bug report.
 If you want to test the update, you can install it with 
 su -c 'yum --enablerepo=updates-testing-newkey update memtest86+'.  You can provide feedback for this update here: http://admin.fedoraproject.org/updates/F9/FEDORA-2009-3526

Comment 12 Fedora Update System 2009-04-27 21:27:37 UTC
memtest86+-2.11-9.fc9 has been pushed to the Fedora 9 testing repository.  If problems still persist, please make note of it in this bug report.
 If you want to test the update, you can install it with 
 su -c 'yum --enablerepo=updates-testing-newkey update memtest86+'.  You can provide feedback for this update here: http://admin.fedoraproject.org/updates/F9/FEDORA-2009-3526

Comment 13 Fedora Update System 2009-04-27 21:28:46 UTC
memtest86+-2.11-9.fc10 has been pushed to the Fedora 10 testing repository.  If problems still persist, please make note of it in this bug report.
 If you want to test the update, you can install it with 
 su -c 'yum --enablerepo=updates-testing update memtest86+'.  You can provide feedback for this update here: http://admin.fedoraproject.org/updates/F10/FEDORA-2009-3511

Comment 14 Fedora Update System 2009-05-12 04:09:44 UTC
memtest86+-2.11-9.fc9 has been pushed to the Fedora 9 stable repository.  If problems still persist, please make note of it in this bug report.

Comment 15 Fedora Update System 2009-05-19 02:05:43 UTC
memtest86+-2.11-9.fc10 has been pushed to the Fedora 10 stable repository.  If problems still persist, please make note of it in this bug report.


Note You need to log in before you can comment on or make changes to this bug.