[Bug 592733] Review Request: turpial - Is cool twitter client with many features and very light
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=592733 --- Comment #25 from Fedora Update System upda...@fedoraproject.org 2011-06-21 13:21:44 EDT --- turpial-1.5.0-2.fc14 has been pushed to the Fedora 14 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 592733] Review Request: turpial - Is cool twitter client with many features and very light
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=592733 Fedora Update System upda...@fedoraproject.org changed: What|Removed |Added Status|ON_QA |CLOSED Fixed In Version||turpial-1.5.0-2.fc14 Resolution||ERRATA Last Closed||2011-06-21 13:21:55 -- 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 592733] Review Request: turpial - Is cool twitter client with many features and very light
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=592733 --- Comment #26 from Fedora Update System upda...@fedoraproject.org 2011-06-21 13:25:25 EDT --- turpial-1.5.0-2.fc15 has been pushed to the Fedora 15 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 592733] Review Request: turpial - Is cool twitter client with many features and very light
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=592733 Fedora Update System upda...@fedoraproject.org changed: What|Removed |Added Fixed In Version|turpial-1.5.0-2.fc14|turpial-1.5.0-2.fc15 -- 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 592733] Review Request: turpial - Is cool twitter client with many features and very light
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=592733 --- Comment #24 from Fedora Update System upda...@fedoraproject.org 2011-06-09 09:24:56 EDT --- turpial-1.5.0-2.fc14 has been submitted as an update for Fedora 14. https://admin.fedoraproject.org/updates/turpial-1.5.0-2.fc14 -- 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 592733] Review Request: turpial - Is cool twitter client with many features and very light
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=592733 --- Comment #23 from Fedora Update System upda...@fedoraproject.org 2011-06-07 15:46:54 EDT --- turpial-1.5.0-2.fc15 has been submitted as an update for Fedora 15. https://admin.fedoraproject.org/updates/turpial-1.5.0-2.fc15 -- 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 592733] Review Request: turpial - Is cool twitter client with many features and very light
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=592733 Edwind Richzendy Contreras Soto richze...@gmail.com changed: What|Removed |Added Flag||fedora-cvs? --- Comment #17 from Edwind Richzendy Contreras Soto richze...@gmail.com 2011-06-04 23:29:20 EDT --- New Package SCM Request === Package Name: turpial Short Description: Turpial is a cool twitter client with many features. support to url short and image hosting services, and more. Owners: richzendy Branches: f14 f15 InitialCC: -- 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 592733] Review Request: turpial - Is cool twitter client with many features and very light
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=592733 Dennis Gilmore den...@ausil.us changed: What|Removed |Added Blocks|177841(FE-NEEDSPONSOR) | AssignedTo|nob...@fedoraproject.org|den...@ausil.us --- Comment #14 from Dennis Gilmore den...@ausil.us 2011-05-27 17:30:14 EDT --- i ill review and sponsor -- 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 592733] Review Request: turpial - Is cool twitter client with many features and very light
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=592733 --- Comment #15 from Edwind Richzendy Contreras Soto richze...@gmail.com 2011-05-27 17:37:07 EDT --- I have a new release: SRC.rpm: http://richzendy.org/repo/turpial/turpial-1.5.0-1.fc15.src.rpm Spec file: http://richzendy.org/repo/turpial/turpial.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 592733] Review Request: turpial - Is cool twitter client with many features and very light
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=592733 --- Comment #16 from Dennis Gilmore den...@ausil.us 2011-05-27 17:48:03 EDT --- builds cleanly in mock, source tarball matches upstream f0c85cc0639287b66210571841920cb35dc9117bce9a3e37616f29b1f1b4768a fedora/SOURCES/turpial-1.5.0.tar.gz f0c85cc0639287b66210571841920cb35dc9117bce9a3e37616f29b1f1b4768a turpial-1.5.0.tar.gz looks good, 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 592733] Review Request: turpial - Is cool twitter client with many features and very light
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=592733 --- Comment #13 from Edwind Richzendy Contreras Soto richze...@gmail.com 2010-08-14 20:19:27 EDT --- I have a new release, spec file in: http://richzendy.org/repo/turpial/turpial.spec The src.rpm it's here: http://richzendy.org/repo/turpial/turpial-1.3.4-1.fc13.src.rpm were implemented all suggestions Chen Lei (Comment 10), Additional suggestions welcome. -- 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 592733] Review Request: turpial - Is cool twitter client with many features and very light
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=592733
--- Comment #12 from Chen Lei supercyp...@gmail.com 2010-08-09 23:57:04 EDT
---
(In reply to comment #11)
Thanks to feed back Chen:
5.
%attr(0755,root,root) %{python_sitelib}/%{name}/main.py
Why add excutable bit for main.py?
This is due to the error message that provides rpmlint:
turpial.noarch: E: non-executable-script
/usr/lib/python2.6/site-packages/turpial/main.py 0644L /usr/bin/python
You should remove shebang from this file instead of adding excutable bit.
See http://fedoraproject.org/wiki/Packaging_tricks#Remove_shebang_from_files
Now is required to place a manual page for each executable? :
turpial.noarch: W: no-manual-page-for-binary turpial
Just ignore this warnings, most rpmlint warnings can be safely ignored.
--
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 592733] Review Request: turpial - Is cool twitter client with many features and very light
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=592733 --- Comment #9 from Edwind Richzendy Contreras Soto richze...@gmail.com 2010-07-25 00:40:55 EDT --- I have a new release, spec file in: http://richzendy.org/repo/turpial/turpial.spec The src.rpm it's here: http://richzendy.org/repo/turpial/turpial-1.3.3-2.fc12.src.rpm rpmlint -i turpial-1.3.3-2.fc12.noarch.rpm 1 packages and 0 specfiles checked; 0 errors, 0 warnings. rpmlint -i turpial-1.3.3-2.fc12.src.rpm 1 packages and 0 specfiles checked; 0 errors, 0 warnings. Any feedback is appreciated. -- 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 592733] Review Request: turpial - Is cool twitter client with many features and very light
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=592733
--- Comment #10 from Chen Lei supercyp...@gmail.com 2010-07-25 00:58:28 EDT
---
Several minor issues:
1.
BuildRoot: %{_tmppath}/%{name}-%{version}-%{release}-root-%(%{__id_u} -n)
%install
rm -rf %{buildroot}
buildroot and cleaning %{buildroot} in %install section is not needed in
fedora.
2.
%setup -q -n turpial-1.3.3
-%setup -q
3.
Summary:Is cool twitter client with many features and very light
-Summary:Cool twitter client with many features and very light
4.
%find_lang %{name}
cat %{name}.lang %{name}-all.lang
Why rename %{name}.lang to %{name}-all.lang?
5.
%attr(0755,root,root) %{python_sitelib}/%{name}/main.py
Why add excutable bit for main.py?
Normally, %attr is not needed in %file. %{python_sitelib}/%{name}/main.py
should not be listed seperately, list files twice in %file is forbidden in
spec.
6.
BuildRequires: python-babel = 0.9.1, python2-devel python-setuptools
Requires: python-simplejson = 1.9.2, python-imaging notify-python =
0.1.1, pygame = 1.7, pygtk2 = 2.12, pywebkitgtk gnome-python2-gtkspell =
2.25.3
Please check the version of those packages in F12, if the version in F12
minimum version, then you should not use versioned requirement.
e.g.
pygtk2 = 2.12 - pygtk2
--
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 592733] Review Request: turpial - Is cool twitter client with many features and very light
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=592733 --- Comment #8 from Edwind Richzendy Contreras Soto richze...@gmail.com 2010-06-19 13:07:08 EDT --- Done, just launched a version of the spec that I have done all the suggestions, if someone can review it would be very grateful: http://richzendy.org/repo/turpial/turpial.spec The src.rpm it's here: http://richzendy.org/repo/turpial/turpial-1.3.3-1.fc12.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 592733] Review Request: turpial - Is cool twitter client with many features and very light
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=592733 --- Comment #5 from Edwind Richzendy Contreras Soto richze...@gmail.com 2010-05-22 19:52:06 EDT --- hi all, here I have an improved version of spec file: http://richzendy.org/repo/turpial/turpial.spec For those wishing to test the rpm, here's to download: http://richzendy.org/repo/turpial/turpial-1.3.2-a1.1.fc12.noarch.rpm For those who want the source rpm, here's to download: http://richzendy.org/repo/turpial/turpial-1.3.2-a1.1.fc12.src.rpm here I have rpmlint output: rpmlint -i turpial-1.3.2-a1.1.fc12.noarch.rpm 1 packages and 0 specfiles checked; 0 errors, 0 warnings. rpmlint -i turpial-1.3.2-a1.1.fc12.src.rpm 1 packages and 0 specfiles checked; 0 errors, 0 warnings. Any recommendation is welcome. -- 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 592733] Review Request: turpial - Is cool twitter client with many features and very light
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=592733
--- Comment #6 from Chen Lei supercyp...@gmail.com 2010-05-22 21:44:27 EDT ---
1.The biggest problem for this package is the location for locale. The right
place for them should be /usr/share/locale/, the upstream must fix them. I
suggest you send a patch to upstream,
/usr/lib/python2.6/site-packages/turpial/i18n is not the appropriate place for
them.
2. the release number is wrong, it should be 0.1.a1
See fedora naming guideline
3.
/usr/share/doc/turpial - /usr/share/doc/turpial-%{version}
4.
/usr/share/applications/turpial.desktop should some further modification.
e.g.
icon- /usr/share/pixmaps/turpial.png
See
http://fedoraproject.org/wiki/Packaging:Guidelines#Desktop_files
http://fedoraproject.org/wiki/Packaging_tricks#.desktop_files
5.
Please add a blank line between the changelog entries
--
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 592733] Review Request: turpial - Is cool twitter client with many features and very light
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=592733 --- Comment #7 from Edwind Richzendy Contreras Soto richze...@gmail.com 2010-05-22 22:27:16 EDT --- Thanks Chen: 1. I have been able to see that locals where in a wrong place, just that I wasn't sure. I report this bug past april-4 and I can help developers to fix the location of this files, because I know that this can improve my spec and define better the languajes macro %find_lang http://code.google.com/p/turpial/issues/detail?id=133 ( in spanish ) 2. Done 3. Done 4. i will make a patch to turpial developers 5. Done where have all the required patches and feedback from developers, i can published another new version of the 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 592733] Review Request: turpial - Is cool twitter client with many features and very light
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=592733 Chen Lei supercyp...@gmail.com changed: What|Removed |Added CC||supercyp...@gmail.com Summary|(FE-NEEDSPONSOR) Review |Review Request: turpial - |Request: turpial - Is cool |Is cool twitter client with |twitter client with many|many features and very |features and very light |light --- Comment #3 from Chen Lei supercyp...@gmail.com 2010-05-16 23:16:32 EDT --- Interesting program. Please follow the guideline below to modify your spec. http://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 592733] Review Request: turpial - Is cool twitter client with many features and very light
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=592733 --- Comment #4 from Edwind Richzendy Contreras Soto richze...@gmail.com 2010-05-16 23:25:12 EDT --- thanks Guilermo and Chei i will work in this feedbacks -- 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
