[Bug 485636] Review Request: cutecom - A GUI application for serial port communications
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=485636 --- Comment #1 from manuel wolfshant 2009-02-16 14:23:30 EDT --- What is the reason of using # Upstream script does not install the .desktop file if KDE is not installed, # so we install it manually: install -D -m 644 $(pwd)/cutecom.desktop ${RPM_BUILD_ROOT}%{_datadir}/applications/cutecom.desktop # Validate .desktop files: desktop-file-validate ${RPM_BUILD_ROOT}%{_datadir}/applications/cutecom.desktop instead of desktop-file-install which install and validates in a sigle step ? According to the source files, the license should be GPLv2+. All of them include: This program is free software; you can redistribute it and/or modify it under the terms of the GNU General Public License as published by the Free Software Foundation; either version 2 of the License, or (at your option) any later version. You must include the license (COPYING) as %doc. I would also include the changelog and the "todo" files but I wouldn't cry if you choose to ignore them. -- 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. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 485636] Review Request: cutecom - A GUI application for serial port communications
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=485636 --- Comment #2 from Jose Luis 2009-02-17 03:55:22 EDT --- You're right Manuel. I've changed it to desktop-file-install. I've also fixed the other issues. This is the new 0.20.0-2 revision: SPEC URL: http://babel.isa.uma.es/mrpt/src-repo/rpm/cutecom.spec SRPM URL: http://babel.isa.uma.es/mrpt/src-repo/rpm/cutecom-0.20.0-2.fc10.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. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 485636] Review Request: cutecom - A GUI application for serial port communications
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=485636 Jose Luis changed: What|Removed |Added AssignedTo|nob...@fedoraproject.org|joseluisblan...@gmail.com -- 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. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 485636] Review Request: cutecom - A GUI application for serial port communications
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=485636 Mamoru Tasaka changed: What|Removed |Added AssignedTo|joseluisblan...@gmail.com |nob...@fedoraproject.org --- Comment #3 from Mamoru Tasaka 2009-02-19 12:47:09 EDT --- Ah... you can't assign what you submitted to yourself. -- 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. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 485636] Review Request: cutecom - A GUI application for serial port communications
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=485636 Mamoru Tasaka changed: What|Removed |Added Status|NEW |ASSIGNED AssignedTo|nob...@fedoraproject.org|mtas...@ioa.s.u-tokyo.ac.jp Flag||fedora-review? --- Comment #4 from Mamoru Tasaka 2009-02-19 13:10:35 EDT --- Assigning to myself. Some notes. * SourceURL - I recomment to use %{version} tag in Source because with this you probably won't have to modify SourceURL when the version is upgraded, ref: https://fedoraproject.org/wiki/Packaging/SourceURL#Using_.25.7Bversion.7D * Description --- It is free software and distributed under the GNU General Public License Version 2, which can find in the file COPYING. --- - is not needed because we can check this by "License" tag in the rebuilt rpm. - I also think that --- It is written using the Qt library by Trolltech. --- is not needed. * Desktop file issue - From build.log --- + desktop-file-install --dir /builddir/build/BUILDROOT/cutecom-0.20.0-2.fc11.i386/usr/share/applications/ /builddir/build/BUILD/cutecom-0.20.0/cutecom.desktop /builddir/build/BUILDROOT/cutecom-0.20.0-2.fc11.i386/usr/share/applications/cutecom.desktop: warning: value "" for key "Path" in group "Desktop Entry" does not look like an absolute path --- I guess Path= item in cutecom.desktop is not needed. - By the way, cutecom.desktop does not have any Categories. Please add the proper one. ? cutecom.desktop specifies "openterm" as Icon, however gnome-icon-theme 2.25.91 does not have openterm.{png,svg} (2.24.x had this icon). Maybe with formal 2.26 gnome-icon-theme release openterm.{png,svg} will reintroduced again, however it may be better that you change Icon item to "utilities-terminal" (actually in gnome-icon-theme 2.24.x, openterm.{png,svg} was symlinks to utilities-terminal.{png,svg}. ? Also please check is the empty line "MimeType=" in cutecom.desktop is needed. If this "MimeType=" line can be removed, then calling "update-desktop-database" on scriptlets is no longer needed, ref: https://fedoraproject.org/wiki/Packaging/ScriptletSnippets#desktop-database -- 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. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 485636] Review Request: cutecom - A GUI application for serial port communications
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=485636 --- Comment #5 from Jose Luis 2009-02-19 17:59:04 EDT --- Thanks Mamoru! > Ah... you can't assign what you submitted to yourself. Okay... wasn't sure about that! > SourceURL Changes to %{name}-%{version} > Description Shortened. > Desktop file. This file has many issues... I've finally left it as: desktop-file-install --remove-key=Path --remove-key=Encoding --remove-key=BinaryPattern --remove-key=TerminalOptions --add-category=System --dir ${RPM_BUILD_ROOT}%{_datadir}/applications/ $(pwd)/cutecom.desktop which fixes almost everything. The only remaining detail is the icon, but I'm not sure on whether it can be changed via desktop-file-install, or I should modify it "manually" (via 'sed', etc...) from the script. The new revision is here: SPEC: http://babel.isa.uma.es/mrpt/src-repo/rpm/cutecom.spec SRPM: http://babel.isa.uma.es/mrpt/src-repo/rpm/cutecom-0.20.0-3.fc10.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. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 485636] Review Request: cutecom - A GUI application for serial port communications
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=485636 Mamoru Tasaka changed: What|Removed |Added Flag|fedora-review? |fedora-review+ --- Comment #6 from Mamoru Tasaka 2009-02-20 11:42:38 EDT --- Looks good. Some comments. - Would you split a long command line into several lines like below so that we can read it easier line below? desktop-file-install \ --remove-key=Path --remove-key=Encoding \ --removekey=BinaryPattern --remove-key=TerminalOptions \ --add-category=System \ --dir ${RPM_BUILD_ROOT}%{_datadir}/applications/ \ $(pwd)/cutecom.desktop (In reply to comment #5) > The only remaining detail is the icon, but I'm > not sure on whether it can be changed via desktop-file-install, or I should > modify it "manually" (via 'sed', etc...) from the script. - You can just use sed (at %prep) This package (cutecom) is APPROVED by mtasaka -- 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. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 485636] Review Request: cutecom - A GUI application for serial port communications
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=485636 Jose Luis changed: What|Removed |Added Flag||fedora-cvs? -- 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. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 485636] Review Request: cutecom - A GUI application for serial port communications
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=485636 --- Comment #7 from Jose Luis 2009-02-22 14:35:19 EDT --- New Package CVS Request === Package Name: cutecom Short Description: A graphical serial terminal. Owners: jlblanco Branches: F-9 F-10 -- 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. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 485636] Review Request: cutecom - A GUI application for serial port communications
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=485636 Kevin Fenzi changed: What|Removed |Added Flag|fedora-cvs? |fedora-cvs+ --- Comment #8 from Kevin Fenzi 2009-02-22 14:43:18 EDT --- cvs done. -- 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. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 485636] Review Request: cutecom - A GUI application for serial port communications
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=485636 --- Comment #9 from Fedora Update System 2009-02-24 15:44:54 EDT --- cutecom-0.20.0-3.fc10 has been pushed to the Fedora 10 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. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 485636] Review Request: cutecom - A GUI application for serial port communications
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=485636 Fedora Update System changed: What|Removed |Added Status|ASSIGNED|CLOSED Fixed In Version||0.20.0-3.fc10 Resolution||NEXTRELEASE -- 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. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review