Overview

Request 613641 accepted

- Fix the wxWidgets requirements. Instead of listing each version
for every *SUSE flavor, we're now calling rpmqpack to find the
correct dependency.
- Update cpanspec.yml accordingly.
- Use %autopatch

Loading...


Christophe Giboudeaux's avatar

@coolo no activity from the package maintainer, do you mind reviewing it ? the current package has installation issues in factory.


Stephan Kulow's avatar

your use of autopatch clearly conflicts with cpanspec. But I'll try to clean this up afterwards


Dirk Stoecker's avatar

Hmm, removing the minimum requires seems not like a good idea. Why do you do that?


Christophe Giboudeaux's avatar

because they're not needed.


Dirk Stoecker's avatar

So the code works with both? Then the comment should also go and the changes entry be a bit more verbose or Factory will reject it.


Dirk Stoecker's avatar

Ah sorry, there is no factory yet. In any case please drop the comment as well and fix cpanspec.yml :-)


Dirk Stoecker's avatar

I go home now - There is a Factory upload!

Other authors of recent changes forgot cpanspec.yml. Please fix it and as said be a bit more verbose about the changelog entry and remove the useless comment in the spec.



Request History
Christophe Giboudeaux's avatar

cgiboudeaux created request

- Fix the wxWidgets requirements. Instead of listing each version
for every *SUSE flavor, we're now calling rpmqpack to find the
correct dependency.
- Update cpanspec.yml accordingly.
- Use %autopatch


Stephan Kulow's avatar

coolo accepted request

openSUSE Build Service is sponsored by