[Bug 1547757] Review Request: python-molecule - Molecule is designed to aid in the development and testing of Ansible roles

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



https://bugzilla.redhat.com/show_bug.cgi?id=1547757



--- Comment #4 from Brett Lentz <blentz@xxxxxxxxxx> ---
> But that's your job as a packager to package the missing dependencies. You'll end up with broken docs, which are an integral part of the package.

I understand your perspective. I'm willing to work on those additional packages
after this package is approved.

Because there is no requirement in the packaging guidelines that all doc and
testing dependencies "must" be met, this issue should not block this package
review.

> Yes but your Python 3 package is behind a conditional, so if you don't build with Python3, you should put the binary inside the Python 2 package.

If you read the spec, the bin is not behind the conditional. If the conditional
is not met, the bin will be included in the python2 package.

> For now it is impractical because it depends on other maintainers to do their job, but as soon as the deps are updated, you should add the tests.

Agreed. My plan is to work on the packaging for these additional dependencies
after this package review is complete.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
_______________________________________________
package-review mailing list -- package-review@xxxxxxxxxxxxxxxxxxxxxxx
To unsubscribe send an email to package-review-leave@xxxxxxxxxxxxxxxxxxxxxxx




[Index of Archives]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Yosemite Conditions]     [KDE Users]

  Powered by Linux