[Bug 567348] Review Request: dreampie - A graphical cross-platform interactive Python shell
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=567348 --- Comment #30 from Robin Lee robinlee.s...@gmail.com 2010-10-23 03:47:34 EDT --- Don't forget to add this package to the comps. Refer to: https://fedoraproject.org/wiki/PackageMaintainers/CompsXml -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 567348] Review Request: dreampie - A graphical cross-platform interactive Python shell
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=567348 --- Comment #29 from Fedora Update System upda...@fedoraproject.org 2010-10-08 16:35:03 EDT --- dreampie-1.1-5.fc13 has been pushed to the Fedora 13 stable repository. If problems still persist, please make note of it in this bug report. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 567348] Review Request: dreampie - A graphical cross-platform interactive Python shell
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=567348 Fedora Update System upda...@fedoraproject.org changed: What|Removed |Added Status|ON_QA |CLOSED Fixed In Version||dreampie-1.1-5.fc13 Resolution||ERRATA Last Closed||2010-10-08 16:35:09 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 567348] Review Request: dreampie - A graphical cross-platform interactive Python shell
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=567348 Fedora Update System upda...@fedoraproject.org changed: What|Removed |Added Status|ASSIGNED|MODIFIED -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 567348] Review Request: dreampie - A graphical cross-platform interactive Python shell
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=567348 --- Comment #27 from Fedora Update System upda...@fedoraproject.org 2010-09-27 13:00:53 EDT --- dreampie-1.1-5.fc13 has been submitted as an update for Fedora 13. https://admin.fedoraproject.org/updates/dreampie-1.1-5.fc13 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 567348] Review Request: dreampie - A graphical cross-platform interactive Python shell
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=567348 Fedora Update System upda...@fedoraproject.org changed: What|Removed |Added Status|MODIFIED|ON_QA --- Comment #28 from Fedora Update System upda...@fedoraproject.org 2010-09-28 01:23:06 EDT --- dreampie-1.1-5.fc13 has been pushed to the Fedora 13 testing repository. If problems still persist, please make note of it in this bug report. If you want to test the update, you can install it with su -c 'yum --enablerepo=updates-testing update dreampie'. You can provide feedback for this update here: https://admin.fedoraproject.org/updates/dreampie-1.1-5.fc13 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 567348] Review Request: dreampie - A graphical cross-platform interactive Python shell
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=567348 --- Comment #26 from Kevin Fenzi ke...@tummy.com 2010-09-26 14:48:09 EDT --- Git done (by process-git-requests). -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 567348] Review Request: dreampie - A graphical cross-platform interactive Python shell
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=567348 Ionuț Arțăriși maple...@lavabit.com changed: What|Removed |Added Flag||fedora-cvs? --- Comment #25 from Ionuț Arțăriși maple...@lavabit.com 2010-09-25 11:04:24 EDT --- New Package SCM Request === Package Name: dreampie Short Description: A graphical cross-platform interactive Python shell Owners: mapleoin Branches: f13 f14 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 567348] Review Request: dreampie - A graphical cross-platform interactive Python shell
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=567348
--- Comment #23 from Ionuț Arțăriși maple...@lavabit.com 2010-09-23 16:29:58
EDT ---
I have replaced the lines to get the python3 version. It seems to work fine now
without the phantom file in %{python3_sitelib}.
http://mapleoin.fedorapeople.org/pkgs/dreampie/dreampie-1.1-5.fc13.src.rpm
http://mapleoin.fedorapeople.org/pkgs/dreampie/dreampie.spec
--
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 567348] Review Request: dreampie - A graphical cross-platform interactive Python shell
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=567348 Robin Lee robinlee.s...@gmail.com changed: What|Removed |Added Flag|fedora-review? |fedora-review+ --- Comment #24 from Robin Lee robinlee.s...@gmail.com 2010-09-24 00:31:35 EDT --- All issues are cleared and build in Rawhide succeeded: http://koji.fedoraproject.org/koji/taskinfo?taskID=2486155 This package is approved by 'cheeselee'. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 567348] Review Request: dreampie - A graphical cross-platform interactive Python shell
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=567348 --- Comment #16 from Ionuț Arțăriși maple...@lavabit.com 2010-09-15 04:27:52 EDT --- Ok, I added all 3 changes. http://mapleoin.fedorapeople.org/pkgs/dreampie/dreampie-1.1-4.fc13.src.rpm http://mapleoin.fedorapeople.org/pkgs/dreampie/dreampie.spec -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 567348] Review Request: dreampie - A graphical cross-platform interactive Python shell
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=567348 Robin Lee robinlee.s...@gmail.com changed: What|Removed |Added Status|NEW |ASSIGNED --- Comment #20 from Robin Lee robinlee.s...@gmail.com 2010-09-15 11:11:09 EDT --- Or better use that tricky file as SOURCE2. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 567348] Review Request: dreampie - A graphical cross-platform interactive Python shell
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=567348
Toshio Ernie Kuratomi a.bad...@gmail.com changed:
What|Removed |Added
CC||a.bad...@gmail.com
--- Comment #21 from Toshio Ernie Kuratomi a.bad...@gmail.com 2010-09-16
00:03:52 EDT ---
I'm sorry for not being clear. Your solution is more complex than it needs to
be. The following three lines will work:
# get python3 version
%global py3_ver 0
%{?__python3: %global py3_ver %(%{__python3} -c import sys;
sys.stdout.write(sys.version[:3]))}
When creating the srpm that's fed to the buildsystem, the spec file needs to be
parsable but it doesn't need to have the correct values. So we need py3_ver to
have a number (in this case 0) so that when it's used in the Requires line
later the spec file will parse correctly.
That srpm is fed to the buildsystem which then creates the binary rpms and a
new srpm. These files are created in a buildroot that has python3-devel
installed and thus py3_ver is set to the proper python3 version when those are
built.
(the other part of your patch to byte compile the things in %{python3_sitelib}
is still necessary.
--
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 567348] Review Request: dreampie - A graphical cross-platform interactive Python shell
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=567348 --- Comment #22 from Robin Lee robinlee.s...@gmail.com 2010-09-16 00:26:11 EDT --- The above solution is excellently better. The requester should follow it and this package will be approved. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 567348] Review Request: dreampie - A graphical cross-platform interactive Python shell
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=567348
--- Comment #13 from Robin Lee robinlee.s...@gmail.com 2010-09-07 08:31:25
EDT ---
* 'rpmlint must be run on every package. The output should be posted in the
review.', quoted from https://fedoraproject.org/wiki/Packaging:ReviewGuidelines
* Add '# The entire source code is GPLv3+ except'
* Relative path should be used to declare license, for example, use
'dreampielib/gui/SimpleGladeApp.py' instead of just the file name
'SimpleGladeApp.py'.
* py_zipimport.py is not found in the source of this release.
* 'LGPLv2.1+' should be 'LGPLv2+'
* BuildRoot tag can be removed.
* md5 matched.
$ md5sum dreampie-1.1.tar.gz
57ced153616069ca6c3c7d37bb30633a dreampie-1.1.tar.gz
* Should instruct users to install dreampie-python3 when they want to use
dreampie with Python3. The proper way is to include a README.Fedora file.
* In this case, subpackage name can be defined through '%package python3'.
* In %description, a comma should be added before 'etc.'
* The summary of the subpackage is not accurate.
* The main rpm should own '%{_datadir}/%{name}/'.
* The subpackage should specify requirement of Python interpreter through
'python(abi)' not 'python3'.
--
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 567348] Review Request: dreampie - A graphical cross-platform interactive Python shell
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=567348 --- Comment #14 from Ionuț Arțăriși maple...@lavabit.com 2010-09-07 12:18:06 EDT --- Thank you, Robin. $ rpmlint dreampie.spec ../SRPMS/dreampie-1.1-3.fc13.src.rpm ../RPMS/noarch/dreampie-1.1-3.fc13.noarch.rpm ../RPMS/noarch/dreampie-python3-1.1-3.fc13.noarch.rpm dreampie.spec: W: no-buildroot-tag dreampie.src: W: no-buildroot-tag dreampie-python3.noarch: W: no-documentation 3 packages and 1 specfiles checked; 0 errors, 3 warnings. Here are the updated files: http://mapleoin.fedorapeople.org/pkgs/dreampie/dreampie.spec http://mapleoin.fedorapeople.org/pkgs/dreampie/dreampie-1.1-3.fc13.src.rpm -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 567348] Review Request: dreampie - A graphical cross-platform interactive Python shell
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=567348
--- Comment #15 from Robin Lee robinlee.s...@gmail.com 2010-09-07 22:03:28
EDT ---
* Upstream README, which you may think just contains useless installation
instructions, should still be contained, I think. It doesn't take much space.
* Don't use hard version of python(abi). Set it at buildtime:
%global py3_ver %(echo `%{__python3} -c import sys;
sys.stdout.write(sys.version[:3])`)
...
Requires: python(abi) = %{py3_ver}
* The subpackage contains just GPLv3+ files. So a License tag should be added
for it.
--
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 567348] Review Request: dreampie - A graphical cross-platform interactive Python shell
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=567348 --- Comment #12 from Ionuț Arțăriși maple...@lavabit.com 2010-09-05 16:23:10 EDT --- Ok, I think that makes sense. http://mapleoin.fedorapeople.org/pkgs/dreampie/dreampie-1.1-2.fc13.src.rpm http://mapleoin.fedorapeople.org/pkgs/dreampie/dreampie.spec -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 567348] Review Request: dreampie - A graphical cross-platform interactive Python shell
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=567348 Robin Lee robinlee.s...@gmail.com changed: What|Removed |Added AssignedTo|nob...@fedoraproject.org|robinlee.s...@gmail.com Flag||fedora-review? -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 567348] Review Request: dreampie - A graphical cross-platform interactive Python shell
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=567348 Robin Lee robinlee.s...@gmail.com changed: What|Removed |Added CC||robinlee.s...@gmail.com --- Comment #8 from Robin Lee robinlee.s...@gmail.com 2010-09-04 13:21:23 EDT --- Is this review on going? A new version has been released upstream. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 567348] Review Request: dreampie - A graphical cross-platform interactive Python shell
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=567348
--- Comment #9 from Robin Lee robinlee.s...@gmail.com 2010-09-04 14:13:50 EDT
---
I also don't think that creating a python3 subpackage for the private modules
used by dreampie itself is a proper practice. People can use dreampie with an
individually compiled Python 3 but without the vendor provided one.
%py_byte_compile %{__python} $RPM_BUILD_ROOT%{python_sitelib}/%{name}lib/
Without the above line, .pyo files are not generated.
Manpages is marked as %doc by default. And explicit %attr setting seems not
needed too.
--
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 567348] Review Request: dreampie - A graphical cross-platform interactive Python shell
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=567348 --- Comment #10 from Ionuț Arțăriși maple...@lavabit.com 2010-09-04 18:13:14 EDT --- Hello Robin, This review is waiting for its reviewer. I also don't think that creating a python3 subpackage for the private modules used by dreampie itself is a proper practice. People can use dreampie with an individually compiled Python 3 but without the vendor provided one. I don't think that's a very good reason. After all, people can use dreampie on a recompiled version of Python 2 or any other of the packages listed as Requires, but that's not a reason to stop using dependencies. The fact remains that those files are useless without a Python3 interpreter and so they need a python3 Requires. The rest of the package is fine without it so that's the reason that the python3 files are segregated in a subpackage, as well as providing extra functionality. I've updated the package and fixed the other two issues, thanks! http://mapleoin.fedorapeople.org/pkgs/dreampie/dreampie-1.1-1.fc13.src.rpm http://mapleoin.fedorapeople.org/pkgs/dreampie/dreampie.spec -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 567348] Review Request: dreampie - A graphical cross-platform interactive Python shell
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=567348 --- Comment #11 from Robin Lee robinlee.s...@gmail.com 2010-09-04 22:41:58 EDT --- The name 'python3-dreampie' implies a DreamPie support for Python3. But in fact, the subpackage is a Python3 support for DreamPie. So it seems better named 'dreampie-python3'. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 567348] Review Request: dreampie - A graphical cross-platform interactive Python shell
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=567348 --- Comment #7 from Ionuț Arțăriși maple...@lavabit.com 2010-07-07 08:04:54 EDT --- Upstream has removed the zip files, so I've made an update: - new upstream version - build separate py3 package - byte-compile py2 and py3 files manually http://mapleoin.fedorapeople.org/pkgs/dreampie/dreampie.spec http://mapleoin.fedorapeople.org/pkgs/dreampie/dreampie-1.0.3-1.fc13.src.rpm -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 567348] Review Request: dreampie - A graphical cross-platform interactive Python shell
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=567348 --- Comment #4 from Ionuț Arțăriși maple...@lavabit.com 2010-03-11 06:29:58 EST --- Hi, Mads I couldn't find a link to the upstream's devel list (I found it now, buried at the bottom of the frontpage). If I had, I could've seen your message in time. On the other hand I think you too should've checked bugzilla or the fedora-package-review list earlier. I agree with your observations, except for the license field, which should probably look like this: License: GPLv3+ and LGPLv3+ and BSD and ASL 2.0 and Python if guidelines were to be followed: https://fedoraproject.org/wiki/Packaging/LicensingGuidelines#Multiple_Licensing_Scenarios Alternatively we could try convincing upstream to relicense some of those files. I was leaving the zip file issue for discussion with the reviewer. I agree that we shouldn't keep them archived, but install them as plain files instead. The py2 files should be installed as plain py files, and the py3 files moved to a py3 subpackage on the archs where it is available. I supppose you mean Fedora releases instead of archs. I don't think there's any reason to create a subpackage since we can't determine at build time if the user has python3 installed and everyone who has python3 will probably want py3 support in dreampie. I think we can just install these data files (as upstream calls them) side-by-side for Fedora releases with python3 (=F13). Running dreampie against either a py2 or py3 interpreter is one of the major features of dreampie imho, so it shouldn't be in a different subpackage. I see you've already established a relationship with upstream, so if you want I can leave this package for you to take ownership of. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 567348] Review Request: dreampie - A graphical cross-platform interactive Python shell
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=567348 --- Comment #5 from Mads Kiilerich m...@kiilerich.com 2010-03-11 09:00:53 EST --- (In reply to comment #4) On the other hand I think you too should've checked bugzilla or the fedora-package-review list earlier. Yeah. I checked before contacting upstream, but I took the wrong semaphore. I should have learned from the dining philosophers and taken all of them. Lesson learned for next time ;-) I see you've already established a relationship with upstream, so if you want I can leave this package for you to take ownership of. Thanks, but no thanks - you came here first, so please keep going. I agree with your observations, except for the license field, which should probably look like this: License: GPLv3+ and LGPLv3+ and BSD and ASL 2.0 and Python if guidelines were to be followed: https://fedoraproject.org/wiki/Packaging/LicensingGuidelines#Multiple_Licensing_Scenarios You are right. But I think that we have to be pragmatic as long as there are no license violations. For example, I think it is fair enough to say that the linux kernel is GPLv2 even though it (AFAIK) contains files under BSD. Alternatively we could try convincing upstream to relicense some of those files. You as packager are free to do it as well. If the package re-license it as GPL then it is GPL. (IANAL) I was leaving the zip file issue for discussion with the reviewer. IMHO the packager should do all the work and review the package himself to make sure it is perfect. The reviewer should just review and verify. But let us discuss it until we agree that the package is perfect and ready for a formal review. I don't think there's any reason to create a subpackage since we can't determine at build time if the user has python3 installed and everyone who has python3 will probably want py3 support in dreampie. I think we can just install these data files (as upstream calls them) side-by-side for Fedora releases with python3 (=F13). Running dreampie against either a py2 or py3 interpreter is one of the major features of dreampie imho, so it shouldn't be in a different subpackage. FWIW I disagree. It is not data files but executable code. If python3 is a build dependency (in fedora versions where it is available) then it _is_ installed. Users who want py3 should install py3 packages. Optional dependencies should be in sub-packages. Nothing special here. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 567348] Review Request: dreampie - A graphical cross-platform interactive Python shell
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=567348 --- Comment #6 from Ionuț Arțăriși maple...@lavabit.com 2010-03-11 16:09:19 EST --- Ok, then I'll keep working on this package, here are the new files: http://mapleoin.fedorapeople.org/pkgs/dreampie/dreampie.spec http://mapleoin.fedorapeople.org/pkgs/dreampie/dreampie-1.0.2-2.fc12.src.rpm * Thu Mar 11 2010 Ionuț C. Arțăriși maple...@fedoraproject.org - 1.0.2-2 - changed group to Development/Tools - marked manpages as %%doc - added pygtk2-libglade dependency - added more license fields: BSD, ASL, Python and Copyright only I talked to tibbs and spot on #fedora-devel and cleared out the licensing issue. The right License tag is the really long and complicated one. One of the motivations for this is that any of those files can be taken and used separately after the package has been installed on a fedora system. FYI, the kernel license is probably explained by the fact that the multiple licensed code is compiled together which is not the issue with python programs. There's more on that here: https://fedoraproject.org/wiki/Licensing/FAQ#Multiple_licensing_situations spot interpreted the whatever you want license as Copyright only so I added that too. I haven't done anything in the spec about the zip files issue yet. I'm going to upstream's mailing list after I finish writing this comment. If python3 is a build dependency (in fedora versions where it is available) then it _is_ installed. Users who want py3 should install py3 packages. I'm not sure what you're saying here. If python3 is a build dependency then it's installed on the build machine, not on the user's machine. Anyway I've thought about this a bit more and I agree. There should be a py3 sub-package which depends on the main package. The rationale that I was getting caught in before was that dreampie running a python3 interpreter still runs on python2 itself. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 567348] Review Request: dreampie - A graphical cross-platform interactive Python shell
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=567348 Mads Kiilerich m...@kiilerich.com changed: What|Removed |Added CC||m...@kiilerich.com, ||methe...@gmail.com --- Comment #3 from Mads Kiilerich m...@kiilerich.com 2010-03-09 20:40:26 EST --- I was working on a package too, but didn't consider it ready for Fedora yet. I was working with upstream while this was submitted without contacting or following upstream. That's a stupid waste of work. Some of my work and review got upstreamed, but please take what you can use from the spec at http://bitbucket.org/kiilerix/dreampie-rpm/src/tip/dreampie.spec - though there isn't much to take except these comments: I would say that one file as LGPLv2+ (which is compatible with and thus relicensable to GPLv3) isn't worth mentioning in the license field. And there are other files with other compatible licenses. Group: It is not a lib but more like Development/Tools The package requires pygtk2-libglade, not just pygtk2 Man pages should be %doc And finally and the reason I got stalled: I think the use of zip files is a show-stopper. It is a ugly hack and doesn't give the transparency we like in Fedora packaging. The py2 files should be installed as plain py files, and the py3 files moved to a py3 subpackage on the archs where it is available. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 567348] Review Request: dreampie - A graphical cross-platform interactive Python shell
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=567348 Fabian Affolter fab...@bernewireless.net changed: What|Removed |Added CC||fab...@bernewireless.net --- Comment #1 from Fabian Affolter fab...@bernewireless.net 2010-03-03 06:45:30 EST --- Just csome quick comments on your spec file: - Is 'CFLAGS=$RPM_OPT_FLAGS' really needed? It's a noarch package. - Please use '%global' instead of '%define' https://fedoraproject.org/wiki/Packaging:Guidelines#.25global_preferred_over_.25define - The README is missing in %doc - The license is GPLv3+, source header says '3 of the License, or (at your option) any later version.' - The latest release is 1.0.2 Please take a look at he following page, too : https://fedoraproject.org/wiki/Packaging:Python -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 567348] Review Request: dreampie - A graphical cross-platform interactive Python shell
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=567348 --- Comment #2 from Ionuț Arțăriși maple...@lavabit.com 2010-03-03 12:36:13 EST --- Thank you, Fabian (In reply to comment #1) Just csome quick comments on your spec file: - Is 'CFLAGS=$RPM_OPT_FLAGS' really needed? It's a noarch package. - Please use '%global' instead of '%define' https://fedoraproject.org/wiki/Packaging:Guidelines#.25global_preferred_over_.25define Done. - The README is missing in %doc I've not included the README because it contains installation instructions for Windows and Debian that I think are unneeded by fedora users. Irrelevant documentation include build instructions, the omnipresent INSTALL file containing generic build instructions, for example, and documentation for non-Linux systems, e.g. README.MSDOS. https://fedoraproject.org/wiki/Packaging/Guidelines#Documentation - The license is GPLv3+, source header says '3 of the License, or (at your option) any later version.' I've updated it and added LGPLv2+ too after finding the build/lib/dreampielib/gui/SimpleGladeApp.py module. - The latest release is 1.0.2 Please take a look at he following page, too : https://fedoraproject.org/wiki/Packaging:Python I have. Could you please be a bit more specific as to what exactly you think is wrong? http://mapleoin.fedorapeople.org/pkgs/dreampie/dreampie.spec http://mapleoin.fedorapeople.org/pkgs/dreampie/dreampie-1.0.2-1.fc12.src.rpm %changelog * Wed Mar 3 2010 Ionuț C. Arțăriși maple...@fedoraproject.org - 1.0.2-1 - fixed spelling error in description - don't make subp_main.py executable - don't pass cflags to build command - use global instead of define macros - changed license field to GPLv3+ and LGPLv2+ - updated to 1.0.2 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
