https://bugzilla.redhat.com/show_bug.cgi?id=951582
Michael Schwendt bugs.mich...@gmx.net changed:
What|Removed |Added
Depends On||1048482
https://bugzilla.redhat.com/show_bug.cgi?id=951582
--- Comment #22 from Michael Schwendt bugs.mich...@gmx.net ---
Since the -static Provides are only for BuildRequires and tracking usage of
static libs, they cannot be arch-specific using %?_isa. It would be
BuildRequires: psi4-static in other
https://bugzilla.redhat.com/show_bug.cgi?id=951582
Bug 951582 depends on bug 1048482, which changed state.
Bug 1048482 Summary: psi4 : does not adhere to Static Library Packaging
Guidelines
https://bugzilla.redhat.com/show_bug.cgi?id=1048482
What|Removed
https://bugzilla.redhat.com/show_bug.cgi?id=951582
--- Comment #23 from Susi Lehtola susi.leht...@iki.fi ---
Ugh, right, the provide doesn't do both. Fixed in rawhide.
--
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this
https://bugzilla.redhat.com/show_bug.cgi?id=951582
Fedora Update System upda...@fedoraproject.org changed:
What|Removed |Added
Status|ON_QA |CLOSED
https://bugzilla.redhat.com/show_bug.cgi?id=951582
Fedora Update System upda...@fedoraproject.org changed:
What|Removed |Added
Fixed In Version|psi4-4.0-0.8.b5.fc19
https://bugzilla.redhat.com/show_bug.cgi?id=951582
--- Comment #19 from Fedora Update System upda...@fedoraproject.org ---
psi4-4.0-0.8.b5.fc19 has been submitted as an update for Fedora 19.
https://admin.fedoraproject.org/updates/psi4-4.0-0.8.b5.fc19
--
You are receiving this mail because:
https://bugzilla.redhat.com/show_bug.cgi?id=951582
--- Comment #20 from Fedora Update System upda...@fedoraproject.org ---
psi4-4.0-0.8.b5.fc20 has been submitted as an update for Fedora 20.
https://admin.fedoraproject.org/updates/psi4-4.0-0.8.b5.fc20
--
You are receiving this mail because:
https://bugzilla.redhat.com/show_bug.cgi?id=951582
Fedora Update System upda...@fedoraproject.org changed:
What|Removed |Added
Status|ASSIGNED|MODIFIED
https://bugzilla.redhat.com/show_bug.cgi?id=951582
Fedora Update System upda...@fedoraproject.org changed:
What|Removed |Added
Status|MODIFIED|ON_QA
https://bugzilla.redhat.com/show_bug.cgi?id=951582
Jens Petersen peter...@redhat.com changed:
What|Removed |Added
Flags|fedora-cvs? |fedora-cvs+
--
https://bugzilla.redhat.com/show_bug.cgi?id=951582
--- Comment #18 from Jens Petersen peter...@redhat.com ---
Git done (by process-git-requests).
--
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
https://bugzilla.redhat.com/show_bug.cgi?id=951582
--- Comment #14 from Susi Lehtola susi.leht...@iki.fi ---
(In reply to Eric Smith from comment #13)
What license covers the files in psi4-data?
A good question. Added LICENSE and COPYING to psi4-data as well.
Anything else?
--
You are
https://bugzilla.redhat.com/show_bug.cgi?id=951582
Eric Smith space...@gmail.com changed:
What|Removed |Added
Flags|fedora-review? |fedora-review+
---
https://bugzilla.redhat.com/show_bug.cgi?id=951582
Susi Lehtola susi.leht...@iki.fi changed:
What|Removed |Added
Flags||fedora-cvs?
---
https://bugzilla.redhat.com/show_bug.cgi?id=951582
--- Comment #17 from Susi Lehtola susi.leht...@iki.fi ---
Thanks for the review! Final spec and srpm
http://theory.physics.helsinki.fi/~jzlehtol/rpms/psi4.spec
http://theory.physics.helsinki.fi/~jzlehtol/rpms/psi4-4.0-0.7.b5.fc20.src.rpm
--
https://bugzilla.redhat.com/show_bug.cgi?id=951582
--- Comment #13 from Eric Smith space...@gmail.com ---
This is a false positive. psi4-data doesn't need the main package.
What license covers the files in psi4-data?
--
You are receiving this mail because:
You are on the CC list for the
https://bugzilla.redhat.com/show_bug.cgi?id=951582
--- Comment #12 from Susi Lehtola susi.leht...@iki.fi ---
(In reply to Eric Smith from comment #11)
Issues:
* Subpackages don't have versioned dependencies
[!]: Fully versioned dependency in subpackages, if present.
Note: No Requires:
https://bugzilla.redhat.com/show_bug.cgi?id=951582
--- Comment #10 from Eric Smith space...@gmail.com ---
fedora-review gets a 404 trying to download your source RPM. Looks like it
should be fc20:
http://theory.physics.helsinki.fi/~jzlehtol/rpms/psi4.spec
https://bugzilla.redhat.com/show_bug.cgi?id=951582
--- Comment #11 from Eric Smith space...@gmail.com ---
Issues:
* Subpackages don't have versioned dependencies
* COPYING isn't in the main package %files
* License tag may need to include zlib/libpng
* Static libraries should be in -static
https://bugzilla.redhat.com/show_bug.cgi?id=951582
--- Comment #9 from Susi Lehtola susi.leht...@iki.fi ---
Added BR and R on numpy.
http://theory.physics.helsinki.fi/~jzlehtol/rpms/psi4.spec
http://theory.physics.helsinki.fi/~jzlehtol/rpms/psi4-4.0-0.5.b5.fc19.src.rpm
Successful koji build
https://bugzilla.redhat.com/show_bug.cgi?id=951582
--- Comment #8 from Susi Lehtola susi.leht...@iki.fi ---
Oh, looks like the new beta was released a month ago... i.e., a lot later than
the week or two I was told in May :P
This one builds on F19 and rawhide.
Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=951582
--- Comment #5 from Eric Smith e...@brouhaha.com ---
Licensecheck reports LGPLv2+ for a bunch of source files in src/lib/libmints.
Maybe needs to be GPLv2+ and LGPLv2+ and MIT.
--
You are receiving this mail because:
You are on
Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=951582
--- Comment #6 from Susi Lehtola susi.leht...@iki.fi ---
(In reply to comment #5)
Licensecheck reports LGPLv2+ for a bunch of source files in
src/lib/libmints. Maybe needs to be GPLv2+ and LGPLv2+ and MIT.
.. but GPLv2+ is
Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=951582
--- Comment #7 from Eric Smith e...@brouhaha.com ---
OK. Just waiting on something that will build on rawhide to complete the
review.
--
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from
Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=951582
--- Comment #3 from Eric Smith e...@brouhaha.com ---
Shouldn't the BuildRequires be python2-devel or python3-devel, rather than just
python-devel?
Could patch the pep8.py to avoid error, and drop patch when new upstream is
used.
Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=951582
--- Comment #4 from Susi Lehtola susi.leht...@iki.fi ---
(In reply to comment #3)
Shouldn't the BuildRequires be python2-devel or python3-devel, rather than
just python-devel?
You're right, changed.
Could patch the pep8.py to
Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=951582
--- Comment #2 from Susi Lehtola susi.leht...@iki.fi ---
(In reply to comment #1)
License tag needs and MIT for /usr/share/psi/python/pep8.py
- Fixed.
/usr/share/psi/python/pep8.py permissions wrong, should be executable?
Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=951582
Eric Smith e...@brouhaha.com changed:
What|Removed |Added
Flags|
Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=951582
--- Comment #1 from Eric Smith e...@brouhaha.com ---
License tag needs and MIT for /usr/share/psi/python/pep8.py
/usr/share/psi/python/pep8.py permissions wrong, should be executable?
Need versioned dependencies in subpackages
Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=951582
Eric Smith e...@brouhaha.com changed:
What|Removed |Added
Status|NEW |ASSIGNED
Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=951582
Susi Lehtola susi.leht...@iki.fi changed:
What|Removed |Added
Alias||psi4
32 matches
Mail list logo