[Bug 192884] Review Request: poedit - GUI editor for GNU gettext .po files
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: poedit - GUI editor for GNU gettext .po files https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=192884 [EMAIL PROTECTED] changed: What|Removed |Added Status|ASSIGNED|CLOSED Resolution||NEXTRELEASE -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug, or are watching the QA contact. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 192884] Review Request: poedit - GUI editor for GNU gettext .po files
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: poedit - GUI editor for GNU gettext .po files https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=192884 [EMAIL PROTECTED] changed: What|Removed |Added Status|NEW |ASSIGNED AssignedTo|[EMAIL PROTECTED] |[EMAIL PROTECTED] OtherBugsDependingO|163776 |163779 nThis|| --- Additional Comments From [EMAIL PROTECTED] 2006-06-20 02:51 EST --- OK - Package name OK - Spec file matches base package name. OK - Meets Packaging Guidelines. OK - License (MIT) OK - License field in spec matches OK - License file included in package OK - Spec in American English OK - Spec is legible. OK - Sources match upstream md5sum: e76a84ae4be4ad9d9a176fd8a1a2effb poedit-1.3.4.tar.gz e76a84ae4be4ad9d9a176fd8a1a2effb poedit-1.3.4.tar.gz.1 OK - Package compiles and builds on at least one arch. n/a - Package needs ExcludeArch OK - BuildRequires correct OK - Spec handles locales/find_lang n/a - Spec has needed ldconfig in post and postun n/a - Package is relocatable and has a reason to be. OK - Package owns all the directories it creates. OK - Package has no duplicate files in %files. OK - Package has %defattr and permissions on files is good. OK - Package has a correct %clean section. OK - Spec has consistant macro usage. OK - Package is code or permissible content. n/a - -doc subpackage needed/used. OK - Packages %doc files don't affect runtime. n/a - Headers/static libs in -devel subpackage. n/a - .pc files in -devel subpackage. n/a - .so files in -devel subpackage. n/a - -devel package Requires: %{name} = %{version}-%{release} n/a - .la files are removed. OK - Package is a GUI app and has a .desktop file OK - Package doesn't own any directories other packages own. See below - No rpmlint output. Issues: 1. Some rpmlint output: Not sure if it's worth fixing these with sed/dos2unix/perl or just reporting it upstream: W: poedit wrong-file-end-of-line-encoding /usr/share/doc/poedit-1.3.4/en/gettext.hhp W: poedit wrong-file-end-of-line-encoding /usr/share/doc/poedit-1.3.4/en/gettext.hhc W: poedit wrong-file-end-of-line-encoding /usr/share/doc/poedit-1.3.4/en/poedit.hhc W: poedit wrong-file-end-of-line-encoding /usr/share/doc/poedit-1.3.4/en/poedit.hhp These are likely locales that rpmlint doesn't know about/are nonstandard? E: poedit incorrect-locale-subdir /usr/share/locale/ast/LC_MESSAGES/poedit.mo E: poedit incorrect-locale-subdir /usr/share/locale/fur/LC_MESSAGES/poedit.mo 2. I did a mock build for fc5 and that worked fine. Given the above comments I also did a rpmbuild --rebuild on both a fc5 and devel machine and those both worked just fine as well, everything seems to build as expected in both mock/non mock env's. 3. There are lots of warnings when building, like: /usr/include/wx-2.6/wx/clntdata.h:31: warning: dereferencing type-punned pointer will break strict-aliasing rules Might be worth reporting upstream. None of the above are blockers, so this package is APPROVED. remember to close this bug with NEXTRELEASE after it's imported and built. -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug, or are watching the QA contact. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 192884] Review Request: poedit - GUI editor for GNU gettext .po files
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: poedit - GUI editor for GNU gettext .po files https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=192884 --- Additional Comments From [EMAIL PROTECTED] 2006-05-24 11:18 EST --- (In reply to comment #5) > I have not missed any buildrequires. Gnome and KDE are NOT required to build > this package -- the configure script will simply install .desktop and icon > files > in a few extra (unnecessary) locations if it finds them (and I delete them > anyway, since they are bogus). > > Okay, what I will do is make sure I rm -rf all potential directories where the > makefile sticks its stuff. That's fine, as long as the build works consistently in environments where those extra files would get installed. -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug, or are watching the QA contact. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 192884] Review Request: poedit - GUI editor for GNU gettext .po files
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: poedit - GUI editor for GNU gettext .po files https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=192884 --- Additional Comments From [EMAIL PROTECTED] 2006-05-24 11:12 EST --- http://blues.mcgill.ca/~icon/fe/poedit-1.3.4-2.src.rpm http://blues.mcgill.ca/~icon/fe/poedit.spec -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug, or are watching the QA contact. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 192884] Review Request: poedit - GUI editor for GNU gettext .po files
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: poedit - GUI editor for GNU gettext .po files https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=192884 --- Additional Comments From [EMAIL PROTECTED] 2006-05-24 10:43 EST --- I have not missed any buildrequires. Gnome and KDE are NOT required to build this package -- the configure script will simply install .desktop and icon files in a few extra (unnecessary) locations if it finds them (and I delete them anyway, since they are bogus). Okay, what I will do is make sure I rm -rf all potential directories where the makefile sticks its stuff. -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug, or are watching the QA contact. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 192884] Review Request: poedit - GUI editor for GNU gettext .po files
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: poedit - GUI editor for GNU gettext .po files https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=192884 --- Additional Comments From [EMAIL PROTECTED] 2006-05-24 09:56 EST --- (In reply to comment #3) > That sounds like a bug to me. Builds should work in any environment with the > appropriate BuildRequires in place. Agreed. A package must build deterministically, no matter which environment is being used. Consider this package BLOCKED. -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug, or are watching the QA contact. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 192884] Review Request: poedit - GUI editor for GNU gettext .po files
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: poedit - GUI editor for GNU gettext .po files https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=192884 --- Additional Comments From [EMAIL PROTECTED] 2006-05-24 09:08 EST --- (In reply to comment #2) > Please build in mock. The build results will always be different depending on your current installation > environment -- thus, the spec file has been written to specifically address building in mock. That sounds like a bug to me. Builds should work in any environment with the appropriate BuildRequires in place. If the files listed in Comment #1 are not installed when building in mock, that suggests that you have missed a build requirement in your spec. -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug, or are watching the QA contact. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 192884] Review Request: poedit - GUI editor for GNU gettext .po files
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: poedit - GUI editor for GNU gettext .po files https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=192884 --- Additional Comments From [EMAIL PROTECTED] 2006-05-24 08:53 EST --- Please build in mock. The build results will always be different depending on your current installation environment -- thus, the spec file has been written to specifically address building in mock. -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug, or are watching the QA contact. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 192884] Review Request: poedit - GUI editor for GNU gettext .po files
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: poedit - GUI editor for GNU gettext .po files https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=192884 --- Additional Comments From [EMAIL PROTECTED] 2006-05-24 01:35 EST --- NEEDSWORK: Checking for unpackaged file(s): /usr/lib/rpm/check-files /var/tmp/poedit-1.3.4-1-root-packman error: Installed (but unpackaged) file(s) found: /usr/share/gnome/apps/Development/poedit.desktop /usr/share/mime-info/poedit.keys /usr/share/mime-info/poedit.mime /usr/share/pixmaps/poedit.png /usr/share/pixmaps/poedit.xpm RPM build errors: Installed (but unpackaged) file(s) found: /usr/share/gnome/apps/Development/poedit.desktop /usr/share/mime-info/poedit.keys /usr/share/mime-info/poedit.mime /usr/share/pixmaps/poedit.png /usr/share/pixmaps/poedit.xpm -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug, or are watching the QA contact. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review