[Bug 1184446] Review Request: tunir - An ultra light CI system
https://bugzilla.redhat.com/show_bug.cgi?id=1184446 --- Comment #11 from Fedora Update System upda...@fedoraproject.org --- tunir-0.6-3.el7 has been submitted as an update for Fedora EPEL 7. https://admin.fedoraproject.org/updates/tunir-0.6-3.el7 -- 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@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1184446] Review Request: tunir - An ultra light CI system
https://bugzilla.redhat.com/show_bug.cgi?id=1184446 Fedora Update System upda...@fedoraproject.org changed: What|Removed |Added Status|ON_QA |CLOSED Fixed In Version||tunir-0.6-2.fc22 Resolution|--- |ERRATA Last Closed||2015-07-17 22:03:37 --- Comment #12 from Fedora Update System upda...@fedoraproject.org --- tunir-0.6-2.fc22 has been pushed to the Fedora 22 stable repository. If problems still persist, please make note of it in this bug report. -- 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@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1184446] Review Request: tunir - An ultra light CI system
https://bugzilla.redhat.com/show_bug.cgi?id=1184446 Fedora Update System upda...@fedoraproject.org changed: What|Removed |Added Status|MODIFIED|ON_QA --- Comment #10 from Fedora Update System upda...@fedoraproject.org --- Package tunir-0.6-2.fc22: * should fix your issue, * was pushed to the Fedora 22 testing repository, * should be available at your local mirror within two days. Update it with: # su -c 'yum update --enablerepo=updates-testing tunir-0.6-2.fc22' as soon as you are able to. Please go to the following url: https://admin.fedoraproject.org/updates/FEDORA-2015-9736/tunir-0.6-2.fc22 then log in and leave karma (feedback). -- 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@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1184446] Review Request: tunir - An ultra light CI system
https://bugzilla.redhat.com/show_bug.cgi?id=1184446 --- Comment #9 from Fedora Update System upda...@fedoraproject.org --- tunir-0.6-2.fc22 has been submitted as an update for Fedora 22. https://admin.fedoraproject.org/updates/tunir-0.6-2.fc22 -- 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@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1184446] Review Request: tunir - An ultra light CI system
https://bugzilla.redhat.com/show_bug.cgi?id=1184446 Fedora Update System upda...@fedoraproject.org changed: What|Removed |Added Status|ASSIGNED|MODIFIED -- 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@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1184446] Review Request: tunir - An ultra light CI system
https://bugzilla.redhat.com/show_bug.cgi?id=1184446 --- Comment #8 from Parag AN(पराग) panem...@gmail.com --- any updates here? can this be closed now? -- 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@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1184446] Review Request: tunir - An ultra light CI system
https://bugzilla.redhat.com/show_bug.cgi?id=1184446 Jon Ciesla limburg...@gmail.com changed: What|Removed |Added Flags|fedora-cvs? |fedora-cvs+ -- 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@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1184446] Review Request: tunir - An ultra light CI system
https://bugzilla.redhat.com/show_bug.cgi?id=1184446 --- Comment #7 from Jon Ciesla limburg...@gmail.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 ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1184446] Review Request: tunir - An ultra light CI system
https://bugzilla.redhat.com/show_bug.cgi?id=1184446 Kushal Das kushal...@gmail.com changed: What|Removed |Added Flags||fedora-cvs? --- Comment #6 from Kushal Das kushal...@gmail.com --- New Package SCM Request === Package Name: tunir Short Description: An ultra light CI system Upstream URL: http://tunir.rtfd.org Owners: kushal Branches: el7 el6 f22 InitialCC: kushal -- 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@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1184446] Review Request: tunir - An ultra light CI system
https://bugzilla.redhat.com/show_bug.cgi?id=1184446 Parag AN(पराग) panem...@gmail.com changed: What|Removed |Added Flags|fedora-review? |fedora-review+ --- Comment #5 from Parag AN(पराग) panem...@gmail.com --- I observed that COPYING and LICENSE files both have same text that is GPLv2+. You may want to drop generic file name COPYING and keep LICENSE in %license You may want to change docker-io to docker as its renamed last week only. This change happened for epel7, F22 and F23 only. You may want to remove unnecessary empty lines between sections. Please fix above issues before importing this package in Fedora. otherwise looks fine. APPROVED. -- 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@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1184446] Review Request: tunir - An ultra light CI system
https://bugzilla.redhat.com/show_bug.cgi?id=1184446 --- Comment #4 from Kushal Das kushal...@gmail.com --- New Spec URL: https://kushal.fedorapeople.org/packages/tunir.spec New SRPM URL: https://kushal.fedorapeople.org/packages/tunir-0.5-1.fc21.src.rpm No Python3 for now. -- 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@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1184446] Review Request: tunir - An ultra light CI system
https://bugzilla.redhat.com/show_bug.cgi?id=1184446 --- Comment #3 from Parag AN(पराग) panem...@gmail.com --- 1) Any plans to add python3 compatibility? 2) License file is not present. Please add license text file and add it using %license macro rest looks good. -- 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@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1184446] Review Request: tunir - An ultra light CI system
https://bugzilla.redhat.com/show_bug.cgi?id=1184446 --- Comment #2 from Kushal Das kushal...@gmail.com --- New Spec URL: https://kushal.fedorapeople.org/packages/tunir.spec New SRPM URL: https://kushal.fedorapeople.org/packages/tunir-0.3-1.fc21.src.rpm -- 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@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1184446] Review Request: tunir - An ultra light CI system
https://bugzilla.redhat.com/show_bug.cgi?id=1184446 Parag AN(पराग) panem...@gmail.com changed: What|Removed |Added Status|NEW |ASSIGNED CC||panem...@gmail.com Assignee|nob...@fedoraproject.org|panem...@gmail.com Flags||fedora-review? -- 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@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1184446] Review Request: tunir - An ultra light CI system
https://bugzilla.redhat.com/show_bug.cgi?id=1184446
--- Comment #1 from Parag AN(पराग) panem...@gmail.com ---
Suggestions:
1) There are many BuildRequires: and Requires: packages on single line, good
to write them as one package per line like
BuildRequires: python2-devel
BuildRequires: python-setuptools
BuildRequires: python-redis
BuildRequires: python-sqlalchemy
Requires: redis
Requires: python-redis
Requires: docker-io
Requires: libguestfs-tools
Requires: python-sqlalchemy
Requires: python-paramiko
2) We now use specific macros python2 or python3, See
http://fedoraproject.org/wiki/Packaging:Python#Macros
3) If this package is not for EPEL5 which I believe not then you can safely
remove Group tag.
4) License is not clear. Only few source files show GPLv2+ whereas PKG-INFO and
setup.py says just GPL and there is no supporting text file of license. Ask
upstream to add license text file.
5) You need to add following in %prep after %setup
# Remove upstream provided egg-info files
rm -rf *.egg-info
6) we already have a macro for /usr/share
%{_prefix}/share/tunir
to
%{_datadir}/tunir
7) Can we have python3 compatibility added in upstream? You know soon we are
moving to use python3 as default.
8) Also, I don't think you need to remove *.py{c,o} files in %install
--
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@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review
