FEDORA-2015-735f979d01 created by tmlcoch 4 years ago for Fedora 22
stable

Several bug fixes

How to install

sudo dnf upgrade --advisory=FEDORA-2015-735f979d01

This update has been submitted for testing by tmlcoch.

4 years ago
User Icon mschwendt commented & provided feedback 4 years ago

The "fix" attempt for bug 1251095 makes my head hurt. By skipping such a broken dependency (i.e. not adding it to the repo metadata!), depsolvers and tools like repoclosure cannot find the unresolvable dep when searching the metadata. Only during RPM's transaction check, the breakage will be discovered. As before. Same symptoms. So, not fixed.

In the ideal case, the repo metadata would not hide the problem, so it could be detected by QA tools.

BZ#1251095 dnf doesn't discover undefined %epoch broken dep
User Icon ngompa commented & provided feedback 4 years ago

While this update does indeed fix the DeltaRPM support and add support for creating xml.xz metadata, I tend to agree with @mschwendt about the way #1251095 was "resolved".

There doesn't appear to be a way for tools built around createrepo_c to correctly handle this issue.

I would consider one of these to be valid solutions:

  • Do not generate the metadata after going through all the packages and returning the error for tools/people to handle. In this case, there would be no metadata for anything to consume because createrepo_c wouldn't make any as long as there are errors in the package data.

  • Add them in anyway, but add some kind of tag indicating that it's a badly written package, and corrections are required. Some repodata validation tool could handle this and return the information appropriately. Likewise, DNF could have code written to handle detection of this particular tag and deal with it differently.

We cannot allow broken dependency chains to exist, in any form.

User Icon ngompa provided feedback 4 years ago
karma
BZ#1251095 dnf doesn't discover undefined %epoch broken dep
BZ#1253850 [rfe] Add --xml-xz to createrepo_c to generate xz compressed XML metadata
BZ#1261031 createrepo_c cannot generate DeltaRPMs even when compiled with drpm
User Icon tmlcoch commented & provided feedback 4 years ago

@mschwendt @ngompa Read my comment about "the bug" here: https://lists.fedoraproject.org/pipermail/devel/2015-August/213882.html Especially the bullet no. 1 and its cons and why the solution you suggest wasn't implemented - createrepo_c tends to be part of critical systems for delivering updates - we just cannot break whole update system and require manual releng action just because some maintainer of some foobar package did a typo in spec of package which is used by two people in the world (where one is the maintainer itself and the second one is a person who installed it by mistake). There is no "problem hiding" - warning is printed. If you are providing invalid input you cannot expect valid output.

User Icon mschwendt commented & provided feedback 4 years ago

If you are able to recognize the input as being invalid, you can exit with error condition. Unfortunately, you choose to skip/ignore input that is detected as being invalid. I've sent a reply to devel@ list.

User Icon tmlcoch commented & provided feedback 4 years ago

@mschwendt, Ideally yes. But createrepo_c is used in places where this would cause problems - see my reply on devel list https://lists.fedoraproject.org/pipermail/devel/2015-October/216015.html

User Icon ngompa commented & provided feedback 4 years ago

@mschwendt, this looks like the undefined epoch thing can't even happen anymore...

User Icon mschwendt commented & provided feedback 4 years ago

Exactly. It's the response to rpmbuild bug 1251453 I had filed. More details in that ticket.

User Icon ngompa commented & provided feedback 4 years ago

In that case, I'll give it a +1 for this update.

BZ#1251095 dnf doesn't discover undefined %epoch broken dep

This update has been pushed to testing.

4 years ago
User Icon yuwata commented & provided feedback 4 years ago
karma

works for me

This update has been submitted for stable by bodhi.

4 years ago
User Icon mhayden commented & provided feedback 4 years ago
karma

Works for me.

This update has been pushed to stable.

4 years ago

Please login to add feedback.

Metadata
Type
enhancement
Karma
3
Signed
Content Type
RPM
Test Gating
Settings
Unstable by Karma
-3
Stable by Karma
3
Dates
submitted
4 years ago
in testing
4 years ago
in stable
4 years ago
BZ#1251095 dnf doesn't discover undefined %epoch broken dep
0
1
BZ#1253850 [rfe] Add --xml-xz to createrepo_c to generate xz compressed XML metadata
0
0
BZ#1261031 createrepo_c cannot generate DeltaRPMs even when compiled with drpm
0
0

Automated Test Results