| Summary: | indexhtml's %post script reads /etc/product.id, but does not require this file b/c of bogus requires | ||
|---|---|---|---|
| Product: | Mageia | Reporter: | Frederik Himpe <fhimpe> |
| Component: | RPM Packages | Assignee: | Malo Deniélou <pmdenielou> |
| Status: | RESOLVED FIXED | QA Contact: | |
| Severity: | normal | ||
| Priority: | release_blocker | CC: | alien, ennael1, mageia, marja11, pmdenielou, rdalverny, thierry.vignaud |
| Version: | Cauldron | Keywords: | Junior_job |
| Target Milestone: | --- | ||
| Hardware: | All | ||
| OS: | Linux | ||
| Whiteboard: | |||
| Source RPM: | indexhtml | CVE: | |
| Status comment: | |||
| Attachments: | full install.log | ||
|
Description
Frederik Himpe
2011-12-29 19:27:29 CET
and was mageia-release-Default installed ? Created attachment 1302 [details]
full install.log
Yes, but only much later during the installation (line 1806: indexhtml, line 2420: mageia-release-Default).
indexhtml should have
Requires(post): mageia-release or something like that.
Yep with the two fix of tmb I understand a little now :) http://svnweb.mageia.org/packages/cauldron/rpm-helper/current/SPECS/rpm-helper.spec?r1=188774&r2=188773&pathrev=188774 /etc/product.id is in mageia-release-Default As there is no maintainer for this package I added the committers in CC. CC:
(none) =>
ahmadsamir3891, ennael1, mageia, rdalverny @ Romain Since you're already working on an indexhtml bug, bug 1306, do you mind taking this one, too? CC:
ahmadsamir3891 =>
marja11 Ok, but... how comes /etc/product.id is not available? (I understand it should be required, but I can't see how it couldn't be available on a properly installed system). As in bug 1306, I'm considering removing the local static page anyway, that will close this issue as a side effect. Status:
NEW =>
ASSIGNED Romain, we are talking about basic packaging and about installing a system. Installer preselect the basesystem package which pulls some other packages in order to have a working system. User select some other packages (either individually or by selecting a desktop). Then urpmi is used to order packages according to their requires. As indexhtml doesn't require the package providing /etc/product.id, it may be installed before that pakage, and thus its post script will fail. The issue is that indexhtml requires mageia-release-common instead of mageia-release. The former doesn't provides /etc/product.id, whereas the later does. Worse you only require it for uninstalling, not for installing (no "requires(post)" on it) It's easily reproducible using a chroot (let's say "I"): mkdir I urpmi.addmedia --urpmi-root=$PWD/I/ /home/tvignaud/mageia/ --distrib urpmi --urpmi-root=$PWD/I/ indexhtml --no-suggests 2>&1|tee LOG.idx (...) 114/114: indexhtml ############################################# cat: /etc/product.id: No such file or directory Keywords:
(none) =>
Junior_job This bug was reassigned to bugsquad, and bugsquad can't solve this, so for now changing status back from assigned to new Status:
ASSIGNED =>
NEW
Thierry Vignaud
2012-04-18 22:45:05 CEST
Status:
NEW =>
ASSIGNED @rda: can you fix this requires in this bug like tv said? so we can resolv this release_blocker... CC:
(none) =>
alien
Malo Deniélou
2012-04-25 23:30:08 CEST
CC:
(none) =>
malo fixed in svn. Please push the package. Done please tester, check it fixes this bug At least urpmi --urpmi-root testcase works smoothly now Status:
ASSIGNED =>
RESOLVED |