[Bug 501099] New: Review Request: perl-XML-Feed - Syndication feed parser and auto-discovery
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. Summary: Review Request: perl-XML-Feed - Syndication feed parser and auto-discovery Alias: perl-XML-Feed https://bugzilla.redhat.com/show_bug.cgi?id=501099 Summary: Review Request: perl-XML-Feed - Syndication feed parser and auto-discovery Product: Fedora Version: rawhide Platform: All URL: http://search.cpan.org/dist/XML-Feed/ OS/Version: Linux Status: NEW Severity: medium Priority: low Component: Package Review AssignedTo: nob...@fedoraproject.org ReportedBy: iarn...@gmail.com QAContact: extras...@fedoraproject.org CC: nott...@redhat.com, fedora-package-review@redhat.com Estimated Hours: 0.0 Classification: Fedora Spec URL: http://fedorapeople.org/~iarnell/review/perl-XML-Feed.spec SRPM URL: http://fedorapeople.org/~iarnell/review/perl-XML-Feed-0.43-1.fc12.src.rpm Description: XML::Feed is a syndication feed parser for both RSS and Atom feeds. It also implements feed auto-discovery for finding feeds, given a URI. Koji build: http://koji.fedoraproject.org/koji/taskinfo?taskID=1357657 *rt-0.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. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 489564] Review Request: Blueman - Bluetooth Manager
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=489564 --- Comment #40 from Michal Ingeli 2009-05-16 03:08:17 EDT --- Two bluetooth icons is not an issue. User can still decide which bluetooth app he/she will use and disable one of them in gnome session manager. Obsoleting package, that provides almost same functionality will force user to choose only one to install. There is no real reason for doing this. -- 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 501101] New: Review Request: emacs-color-theme - elisp mode to customize emacs look and feel
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. Summary: Review Request: emacs-color-theme - elisp mode to customize emacs look and feel https://bugzilla.redhat.com/show_bug.cgi?id=501101 Summary: Review Request: emacs-color-theme - elisp mode to customize emacs look and feel Product: Fedora Version: rawhide Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: low Component: Package Review AssignedTo: nob...@fedoraproject.org ReportedBy: fargio...@gnome.org QAContact: extras...@fedoraproject.org CC: nott...@redhat.com, fedora-package-review@redhat.com Estimated Hours: 0.0 Classification: Fedora Spec URL: http://people.freedesktop.org/~fargiolas/color-theme/emacs-color-theme.spec SRPM URL: http://people.freedesktop.org/~fargiolas/color-theme/emacs-color-theme-6.6.0-1.fc11.src.rpm Description: color-theme is an emacs mode that provides a lot of different presets to customize emacs faces. It also includes a little framework to easily create new themes from your own customizations and submit them to the maintainer. Check http://www.nongnu.org/color-theme/ and http://www.emacswiki.org/emacs/ColorTheme for more informations. This is my first package, usually I'm not too much interested into packaging stuff, but this is one of the first things I always install right after the distribution, so it shouldn't be hard to keep the spec file up to date. Anyway, I'm also fine if someone else wants to take its maintainership for me (it's pretty trivial, I'm not sure it's worth to go into the whole sponsorship, account requesting thing just for this tiny package :P). -- 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 498748] Review Request: rpmdepsize - A tool for visualizing RPM dependencies
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=498748 --- Comment #11 from Fedora Update System 2009-05-16 04:14:00 EDT --- rpmdepsize-1.0-3.fc11 has been submitted as an update for Fedora 11. http://admin.fedoraproject.org/updates/rpmdepsize-1.0-3.fc11 -- 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 498748] Review Request: rpmdepsize - A tool for visualizing RPM dependencies
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=498748 --- Comment #10 from Fedora Update System 2009-05-16 04:11:56 EDT --- rpmdepsize-1.0-3.fc10 has been submitted as an update for Fedora 10. http://admin.fedoraproject.org/updates/rpmdepsize-1.0-3.fc10 -- 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 496880] Review Request: wannier90 - Maximally-localised Wannier functions
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=496880 --- Comment #12 from Fedora Update System 2009-05-16 04:23:51 EDT --- wannier90-1.1-4.fc11 has been submitted as an update for Fedora 11. http://admin.fedoraproject.org/updates/wannier90-1.1-4.fc11 -- 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 496880] Review Request: wannier90 - Maximally-localised Wannier functions
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=496880 --- Comment #11 from Fedora Update System 2009-05-16 04:22:36 EDT --- wannier90-1.1-4.fc10 has been submitted as an update for Fedora 10. http://admin.fedoraproject.org/updates/wannier90-1.1-4.fc10 -- 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 456353] Review Request: libffado - Free firewire audio driver library
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=456353 --- Comment #7 from John Haxby 2009-05-16 04:43:01 EDT --- 2.0-rc1 is available now. Is there any chance of this making it into F11? -- 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 225617] Merge Review: bitmap-fonts
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=225617 --- Comment #10 from Nicolas Mailhot 2009-05-16 05:25:26 EDT --- This package is currently a bundle of fonts collected from different sources with different licenses. Our font packaging guidelines would call for some splitting here (terminus-fonts is an example of mono-bitmap-font package) -- 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 501098] Review Request: perl-XML-RSS-LibXML - XML::RSS with XML::LibXML
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=501098 Parag AN(पराग) changed: What|Removed |Added Status|NEW |ASSIGNED CC||panem...@gmail.com AssignedTo|nob...@fedoraproject.org|panem...@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. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 501099] Review Request: perl-XML-Feed - Syndication feed parser and auto-discovery
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=501099 Parag AN(पराग) changed: What|Removed |Added Status|NEW |ASSIGNED CC||panem...@gmail.com AssignedTo|nob...@fedoraproject.org|panem...@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. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 498736] Review Request: ucommon - Portable C++ runtime for threads and sockets
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=498736 --- Comment #27 from David Sugar 2009-05-16 07:36:26 EDT --- checking now. If it builds, do I do a "make update" in devel to push it through so I can build sipwitch? I recall seeing NOT to do so for devel here (http://fedoraproject.org/wiki/PackageMaintainers/Join#Check_out_the_module), but if not, then how is it made available as a dependency for building sipwitch in rawhide? -- 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 498736] Review Request: ucommon - Portable C++ runtime for threads and sockets
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=498736 --- Comment #28 from David Sugar 2009-05-16 08:07:21 EDT --- Never mind. sipwitch built fine too now in rawhide/f12. -- 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 488174] Review Request: nimbus-theme-gnome - The Nimbus theme originally from Sun
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=488174 Matej Cepl changed: What|Removed |Added Flag||needinfo+ --- Comment #9 from Matej Cepl 2009-05-16 08:18:56 EDT --- Tried to fix all %files and subpackages issues, but I feel pretty lost in this. Please, take a look whether my current solution makes sense. (In reply to comment #7) > Regarding index.theme: Is there a nimbus-notification theme? Not sure about that, cannot find anything. Will check > The license is not clear. Is it LGPLv2 or LGPLv2+? Can you ask upstream for > clarification please? I would say that in doubts I would keep it as strict LGPLv2. I will ask upstream (first I have to ask who actually is upstream maintainer). What do you think? -- 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 488174] Review Request: nimbus-theme-gnome - The Nimbus theme originally from Sun
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=488174 Matej Cepl changed: What|Removed |Added Flag|needinfo+ | -- 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 488174] Review Request: nimbus-theme-gnome - The Nimbus theme originally from Sun
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=488174 Matej Cepl changed: What|Removed |Added Flag||needinfo?(fed...@christoph- ||wickert.de) -- 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 488174] Review Request: nimbus-theme-gnome - The Nimbus theme originally from Sun
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=488174 --- Comment #10 from Matej Cepl 2009-05-16 08:25:02 EDT --- Scratch build went successfully http://koji.fedoraproject.org/koji/taskinfo?taskID=1357884 -- 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 488174] Review Request: nimbus-theme-gnome - The Nimbus theme originally from Sun
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=488174 --- Comment #11 from Matej Cepl 2009-05-16 08:45:28 EDT --- Created an attachment (id=344264) --> (https://bugzilla.redhat.com/attachment.cgi?id=344264) modfiied .spec file -- 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 488174] Review Request: nimbus-theme-gnome - The Nimbus theme originally from Sun
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=488174 --- Comment #12 from Matej Cepl 2009-05-16 08:46:40 EDT --- Sorry, previous scratch-build is unbuildable (typo in Requires). The real scratch build is http://koji.fedoraproject.org/koji/taskinfo?taskID=1357946 (src.rpm is http://koji.fedoraproject.org/scratch/mcepl/task_1357946/nimbus-0.0.17-4.fc11.src.rpm) and the last spec file has been attached to this bug. -- 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 497441] Review Request: mumble - Voice chat application
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=497441
--- Comment #60 from Igor Jurišković 2009-05-16
09:21:53 EDT ---
Tnx for reply Mamoru.
(In reply to comment #59)
> Well, I have not fully read previous comments, however
> for -9:
>
> * BR (BuildRequires)
> - Would you explain why non-devel packages like:
> ---
> dbus-qt
> qt-sqlite
> ---
> have to be written as BRs explicitly?
>
There is a problem. qt-sqlite is not available on F10. Because of that(i think
cause of that) mumble fails to build at koji. What is the replacement for
qt-sqlite on F10? I was searching but found nothing.
> * Requires
> - Proper Requires(pre) or so are missing:
> https://fedoraproject.org/wiki/Packaging/UsersAndGroups
>
Done.
> https://fedoraproject.org/wiki/Packaging/SysVInitScript#InitscriptScriptlets
> - Would you explain why murmur subpackage does not depend
> on mumble main package?
You dont need mumble to run murmur. Murmur is for servers and mumble is for
clients.
> - The directory %{_datadir}/kde4/services/ is owned by kde-filesystem
> so it is better that -protocol subpackage should have
> "Requires: kde-filesystem".
>
Done.
> * Parallel make
> - Support parallel make if possible. If not possible
> write some comments about this:
> https://fedoraproject.org/wiki/Packaging/Guidelines#Parallel_make
>
Done.
> * Installation directory
> --
> install -pD -m0755 release/murmurd %{buildroot}%{_sbindir}/murmurd
> ln -s ../sbin/murmurd %{buildroot}%{_bindir}/%{name}-server
> --
> - Why is the same command has to be in two different directories
> in the path with different names?
Less confusion. After years of mumble nobody told me murmur - everyone says
mumble-server.
>
> - By the way
> --
> %attr(-,mumble-server,mumble-server) %{_bindir}/mumble-server
> --
> Here %{_bindir}/mumble-server is a symlink and
> %attr(.) on symlink is useless (see man symlink(2))
> ! By the way using %{name} macro is preferred because
> you already use it.
>
Fixed.
> --
> #man pages
> mkdir -p %{buildroot}%{_mandir}/
> install -pD -m0664 man/murmurd.1 %{buildroot}%{_mandir}/
> install -pD -m0664 man/mumble* %{buildroot}%{_mandir}/
> --
> - These man pages must be installed under %_mandir/man1,
> not under %_mandir (and also see below)
>
Done.
> * Permissions
> --
> install -pD -m0664 icons/%{name}.16x16.png
> %{buildroot}%{_datadir}/icons/hicolor/16x16/apps/%{name}.png
> --
> - Usually these files should have 0644 (not 0664) permission
>
Done.
> --
> install -pD scripts/%{name}.protocol
> %{buildroot}%{_datadir}/kde4/services/%{name}.protocol
> install -pD scripts/murmur.conf
> %{buildroot}%{_sysconfdir}/dbus-1/system.d/murmur.conf
> --
> - Due to this these files have executable permission however it
> seems these should be with 0644 permision.
>
Done.
> * Inconsistent commands
> --
> install -d %{buildroot}%{_libdir}/%{name}/
> mkdir -p %{buildroot}%{_mandir}/
> --
> - Why do you use both "install -d" and "mkdir -p"?
>
Fixed.
> * Scriptlets
> - Scriptlets for GTK cache updating does not match current Fedora
> guideline. Please fix this:
> https://fedoraproject.org/wiki/Packaging/ScriptletSnippets#Icon_Cache
> (especially check %postun)
>
Done.
> - Please specify the home directory of mumble-server user
> (see the -d option:
> https://fedoraproject.org/wiki/Packaging/UsersAndGroups )
> Currently this is set as "/home/mumble-server" automatically
> (on Fedora)
>
What should set as home directory? It says directory with data but there is
none.
> --
> %preun -n murmur
> if [$1 = 0]; then
> --
> - This is wrong.
> --
> [tasa...@localhost ~]$ num=0
> [tasa...@localhost ~]$ if [$num = 0] ; then echo "yes" ; fi
> -bash: [0:
[Bug 498736] Review Request: ucommon - Portable C++ runtime for threads and sockets
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=498736 --- Comment #29 from Mamoru Tasaka 2009-05-16 09:50:25 EDT --- Okay, thank you. -- 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 499137] Review Request: sipwitch - SIP telephony server for secure phone systems
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=499137 Mamoru Tasaka changed: What|Removed |Added Status|ASSIGNED|CLOSED Resolution||NEXTRELEASE Bug 499137 depends on bug 498736, which changed state. Bug 498736 Summary: Review Request: ucommon - Portable C++ runtime for threads and sockets https://bugzilla.redhat.com/show_bug.cgi?id=498736 What|Old Value |New Value Status|ASSIGNED|CLOSED Resolution||NEXTRELEASE --- Comment #19 from Mamoru Tasaka 2009-05-16 09:50:33 EDT --- Now closing. -- 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 500476] Review Request: cups-bjnp- cups backend for Canon bjnp (USB over IP) printers
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=500476 --- Comment #2 from Louis Lagendijk 2009-05-16 11:16:15 EDT --- Updated version: Spec URL: http://www.fazant.net/cups-bjnp/cups-bjnp.spec SRPM URL: http://www.fazant.net/cups-bjnp/cups-bjnp-0.5.2-2.fc10.src.rpm - removed superfluous imake dependency -- 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 487114] Review Request: gvrpcd - A program for announcing VLANs using GVRP.
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=487114 --- Comment #21 from Fedora Update System 2009-05-16 11:24:51 EDT --- gvrpcd-1.3-2.fc11 has been submitted as an update for Fedora 11. http://admin.fedoraproject.org/updates/gvrpcd-1.3-2.fc11 -- 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 487114] Review Request: gvrpcd - A program for announcing VLANs using GVRP.
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=487114 --- Comment #22 from Fedora Update System 2009-05-16 11:25:58 EDT --- gvrpcd-1.3-2.fc10 has been submitted as an update for Fedora 10. http://admin.fedoraproject.org/updates/gvrpcd-1.3-2.fc10 -- 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 501116] New: Review Request: perl-HTML-CalendarMonthSimple - Perl Module for Generating HTML Calendars
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. Summary: Review Request: perl-HTML-CalendarMonthSimple - Perl Module for Generating HTML Calendars https://bugzilla.redhat.com/show_bug.cgi?id=501116 Summary: Review Request: perl-HTML-CalendarMonthSimple - Perl Module for Generating HTML Calendars Product: Fedora Version: rawhide Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: medium Component: Package Review AssignedTo: nob...@fedoraproject.org ReportedBy: xav...@bachelot.org QAContact: extras...@fedoraproject.org CC: nott...@redhat.com, fedora-package-review@redhat.com Estimated Hours: 0.0 Classification: Fedora Spec URL: http://www.bachelot.org/fedora/SPECS/perl-HTML-CalendarMonthSimple.spec SRPM URL: http://www.bachelot.org/fedora/SRPMS/perl-HTML-CalendarMonthSimple-1.25-2.fc10.src.rpm Description: HTML::CalendarMonthSimple is a Perl module for generating, manipulating, and printing a HTML calendar grid for a specified month. It is intended as a faster and easier-to-use alternative to HTML::CalendarMonth. -- 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 501116] Review Request: perl-HTML-CalendarMonthSimple - Perl Module for Generating HTML Calendars
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=501116 --- Comment #1 from Xavier Bachelot 2009-05-16 11:30:16 EDT --- I'm not sure of the license. Upstream says : " This Perl module is freeware. It may be copied, derived, used, and distributed without limitation." (http://search.cpan.org/~stigmata/HTML-CalendarMonthSimple-1.25/CalendarMonthSimple.pm#AUTHORS,_CREDITS,_COPYRIGHTS). I translated that to Public Domain. -- 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 497441] Review Request: mumble - Voice chat application
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=497441 Mamoru Tasaka changed: What|Removed |Added Status|NEW |ASSIGNED AssignedTo|nob...@fedoraproject.org|mtas...@ioa.s.u-tokyo.ac.jp 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. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 497441] Review Request: mumble - Voice chat application
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=497441 --- Comment #61 from Kevin Kofler 2009-05-16 12:51:55 EDT --- SQLite support is now included in the main qt package because it's used by the new help system in Qt >= 4.4. -- 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 494199] Review Request: drascula-international - Subtitles for Drascula: The Vampire Strikes Back
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=494199 Felix Kaechele changed: What|Removed |Added Flag|fedora-review? |fedora-review+ --- Comment #6 from Felix Kaechele 2009-05-16 13:21:25 EDT --- Okay. That seems to fit. This package is APPROVED! I will of course also review your drascula-music package. Regarding the openttd package: grfcodec is now in Fedora. I was preparing for my final exams in school so I was somewhat lacking time. But now I have loads of free time to kill :) That means openttd will be in Fedora pretty soon. -- 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 494197] Review Request: drascula-music - Background music for Drascula: The Vampire Strikes Back
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=494197 --- Comment #5 from Felix Kaechele 2009-05-16 13:27:05 EDT --- The following items have been checked and are ok: 1. rpmlint is silent 2. package name complies to guidelines 3. package meets packaging guidelines 4. sha1sums match: [fe...@polaris SOURCES]$ sha1sum drascula-audio-1.0.zip* 6618ebe2f08f94836262bc68de6022f9d8f9b081 drascula-audio-1.0.zip 6618ebe2f08f94836262bc68de6022f9d8f9b081 drascula-audio-1.0.zip.orig 5. the spec file is really simple and legible even by my 80-year old grandma 6. the package builds great because it's noarch :-) 7. Requires and BuildReqs are sane 8. file ownership is ok 9. macro usage is reasonable 10. package contains content 11. doesn't own stuff it shouldn't own The following items need to be addressed: 1. Is this really GPLv2+? the readme doesn't state so. Maybe the same mistake as in the drascula-international package? -- 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 501006] Review Request: xine-ui - A skinned xlib-based gui for xine-lib
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=501006 --- Comment #3 from Zarko (grof) 2009-05-16 13:33:53 EDT --- Is this xine-ui.spec taken from RPM Fusion? If it is, then why we must reviewing again something what is reviewed earlier? -- 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 494199] Review Request: drascula-international - Subtitles for Drascula: The Vampire Strikes Back
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=494199
Christoph Wickert changed:
What|Removed |Added
CC||fed...@christoph-wickert.de
--- Comment #7 from Christoph Wickert 2009-05-16
13:42:03 EDT ---
Isn't %{_datadir}/drascula/ unowned? The package has no requirements on
something that owns %{_datadir}/drascula/ (in fact it has no requirements at
all). I guess "Requires: drascula" is missing, right?
--
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 494199] Review Request: drascula-international - Subtitles for Drascula: The Vampire Strikes Back
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=494199 Christoph Wickert changed: What|Removed |Added CC|fed...@christoph-wickert.de | --- Comment #8 from Christoph Wickert 2009-05-16 13:49:31 EDT --- Sorry for the above comment, was ether a firefox or pebcak error. Searched the spec for "Requires" and FF gave me no results. -- 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 501006] Review Request: xine-ui - A skinned xlib-based gui for xine-lib
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=501006 --- Comment #4 from Mamoru Tasaka 2009-05-16 14:00:57 EDT --- Because Fedora and rpmfusion are different... -- 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 491519] Review Request: openttd-opengfx - OpenGFX replacement graphics for OpenTTD
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=491519 Bug 491519 depends on bug 498744, which changed state. Bug 498744 Summary: Review Request: grfcodec - A suite of programs to modify Transport Tycoon Deluxe's GRF files https://bugzilla.redhat.com/show_bug.cgi?id=498744 What|Old Value |New Value Status|NEW |ASSIGNED Resolution||NEXTRELEASE Status|ASSIGNED|CLOSED --- Comment #17 from Felix Kaechele 2009-05-16 14:02:16 EDT --- It builds from source now: http://koji.fedoraproject.org/koji/taskinfo?taskID=1358203 New SPEC: http://heffer.fedorapeople.org/review/openttd-opengfx.spec New SRPM: http://heffer.fedorapeople.org/review/openttd-opengfx-0-0.3.alpha4.2.fc11.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 497441] Review Request: mumble - Voice chat application
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=497441
--- Comment #62 from Mamoru Tasaka 2009-05-16
14:10:56 EDT ---
Created an attachment (id=344291)
--> (https://bugzilla.redhat.com/attachment.cgi?id=344291)
Patch to compile on dist-f10-updates-candidate
For -10:
* BR
- BR: qt3-sqlite should not be needed. This uses Qt4, not
Qt3.
* About mumble-server user directory
- Well, -d %{buildroot}%{_datadir}/%{name}/ is definitely wrong,
this directory should be used only when building this package.
I would suggest -d %{_localstatedir}/lib/mumble-server/
* Requires
- Needed Requires(pre) or so are still missing for murmur subpackage:
https://fedoraproject.org/wiki/Packaging/SysVInitScript#Initscripts_in_spec_file_scriptlets
* Build failure
- The attached patch will make this package compile on
F-11/F-10 (F-12 not checked)
http://koji.fedoraproject.org/koji/taskinfo?taskID=1358131
http://koji.fedoraproject.org/koji/taskinfo?taskID=1358127
* Devel files in main package
--
ln -s %{_libdir}/libmumble.so.%{version} %{buildroot}%{_libdir}/libmumble.so
--
- I guess %_libdir/libmumble.so is not needed.
Note that rpmlint warns about this:
--
mumble.i586: W: devel-file-in-non-devel-package /usr/lib/libmumble.so
--
* Service enabled by default
--
murmur.i586: W: service-default-enabled /etc/rc.d/init.d/murmur
--
- Installing murmur package enables "murmur" service by default
on level 345:
--
# chkconfig --list murmur
murmur 0:off 1:off 2:off 3:on 4:on 5:on 6:off
--
These should be disabled by default.
In murmur init script, # Default-Start: item should have no
level.
* Init script fix
--
diff -uN mumble-1.1.8-9.fc9.src/murmur.init
mumble-1.1.8-10.fc10.src/murmur.init
--- mumble-1.1.8-9.fc9.src/murmur.init 2009-04-30 01:03:18.0 +0900
+++ mumble-1.1.8-10.fc10.src/murmur.init2009-05-15 08:32:48.0
+0900
@@ -52,7 +52,7 @@
RETVAL=$?
echo
if [ $RETVAL -eq 0 ]; then
- /bin/rm -f "$lockfile" "$pidfile" 2> /dev/null && success ||
failure
+ /bin/rm -f "$lockfile" "$pidfile" 2> /dev/null || failure
fi
}
--
- I guess " || failure" can also be removed.
* undefined non-weak symbols
--
mumble.i586: W: undefined-non-weak-symbol /usr/lib/libmumble.so.1.1.8
glAttachShader
mumble.i586: W: undefined-non-weak-symbol /usr/lib/libmumble.so.1.1.8 glEnable
mumble.i586: W: undefined-non-weak-symbol /usr/lib/libmumble.so.1.1.8
glTexCoordPointer
(and so many)
--
- libmumble.so has undefined non-weak symbols. As
this package does not provide -devel subpackage, currently this
can be ignored, however it is preferable that this issue
is fixed.
! You can also check this issue by
$ ldd -r /usr/lib/libmumble.so.1 > /dev/null
This can be fixed by making libmumble.so linked against libGL.so.
* Permission
--
install -pD -m0664 man/murmurd.1 %{buildroot}%{_mandir}/man1/
install -pD -m0664 man/mumble* %{buildroot}%{_mandir}/man1/
--
- These should be 0644 permission
(In reply to comment #60)
> > * Installation directory
> > --
> > install -pD -m0755 release/murmurd %{buildroot}%{_sbindir}/murmurd
> > ln -s ../sbin/murmurd %{buildroot}%{_bindir}/%{name}-server
> > --
> > - Why is the same command has to be in two different directories
> > in the path with different names?
> Less confusion. After years of mumble nobody told me murmur - everyone says
> mumble-server.
- Well, What I wonder is that where murmurd is under /usr/sbin,
mumble-server is under /usr/bin. Why these directories are different?
--
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.
__
[Bug 501006] Review Request: xine-ui - A skinned xlib-based gui for xine-lib
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=501006 Mamoru Tasaka changed: What|Removed |Added Status|NEW |ASSIGNED AssignedTo|nob...@fedoraproject.org|mtas...@ioa.s.u-tokyo.ac.jp Flag||fedora-review? --- Comment #5 from Mamoru Tasaka 2009-05-16 14:19:47 EDT --- Assigning. -- 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 497441] Review Request: mumble - Voice chat application
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=497441 --- Comment #63 from Thorvald Natvig 2009-05-16 14:24:03 EDT --- (In reply to comment #62) > * undefined non-weak symbols > -- > mumble.i586: W: undefined-non-weak-symbol /usr/lib/libmumble.so.1.1.8 > glAttachShader > mumble.i586: W: undefined-non-weak-symbol /usr/lib/libmumble.so.1.1.8 glEnable > mumble.i586: W: undefined-non-weak-symbol /usr/lib/libmumble.so.1.1.8 > glTexCoordPointer > libmumble.so is an injection library to get the overlay in third-party applications. It can't link (and hence load) libGL on it's own, as this breaks games that provide their own GL wrapper. -- 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 501006] Review Request: xine-ui - A skinned xlib-based gui for xine-lib
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=501006 --- Comment #6 from Mamoru Tasaka 2009-05-16 14:41:09 EDT --- For -8: * Desktop file - Remove "Application" category. This category is marked as deprecated. ref: http://www.redhat.com/archives/fedora-extras-list/2006-October/msg00723.html * desktop-file-install - Again "--remove-original" is not needed for this case. ?? About aaxine - Well, currently launching aaxine on CUI (with framebuffer, vga=791) by $ aaxine just displays nothing and only sound can be heard. Also with GUI launching aaxine on terminal emulator (such as gnome-terminal) launching another Window and show some AA images there, not on the terminal emulator. Is this the expected behavior? -- 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 501006] Review Request: xine-ui - A skinned xlib-based gui for xine-lib
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=501006 --- Comment #7 from Jussi Lehtola 2009-05-16 15:20:25 EDT --- (In reply to comment #6) > For -8: > > * Desktop file > - Remove "Application" category. This category is marked as > deprecated. ref: > > http://www.redhat.com/archives/fedora-extras-list/2006-October/msg00723.html Removed. > * desktop-file-install > - Again "--remove-original" is not needed for this case. Sorry, didn't see it since it was on the same line as desktop-install :) Removed. > ?? About aaxine > - Well, currently launching aaxine on CUI (with framebuffer, vga=791) > by $ aaxine > just displays nothing and only sound can be heard. > Also with GUI launching aaxine on terminal emulator (such as > gnome-terminal) launching another Window and show some AA images > there, not on the terminal emulator. > Is this the expected behavior? After toying around for some time I found out that aalib output requires support from xine-lib. This is in the xine-lib-extras package, which I have now added as a Requires: and aaxine works. http://theory.physics.helsinki.fi/~jzlehtol/rpms/xine-ui.spec http://theory.physics.helsinki.fi/~jzlehtol/rpms/xine-ui-0.99.5-9.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 501006] Review Request: xine-ui - A skinned xlib-based gui for xine-lib
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=501006 --- Comment #8 from Zarko (grof) 2009-05-16 15:41:11 EDT --- Im getting this issue when pressing DVD or DVB navigator buttons: cannot find input plugin for MRL [dvd:/] input plugin cannot open MRL [dvd:/] but I did not try the latest buid. -- 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 501006] Review Request: xine-ui - A skinned xlib-based gui for xine-lib
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=501006 --- Comment #9 from Zarko (grof) 2009-05-16 15:50:39 EDT --- Is this OK (at config time)? checking for NVTVSIMPLE... *** nvtvsimple support will be disabled *** -- 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 501006] Review Request: xine-ui - A skinned xlib-based gui for xine-lib
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=501006 --- Comment #10 from Jussi Lehtola 2009-05-16 16:13:05 EDT --- (In reply to comment #8) > Im getting this issue when pressing DVD or DVB navigator buttons: > > cannot find input plugin for MRL [dvd:/] > input plugin cannot open MRL [dvd:/] > > but I did not try the latest buid. For dvb you need to have a working channels.conf in your ~/.xine directory. For dvd you must have a dvd in your drive. Otherwise you get these errors. Stupid, yes. A packaging bug, no. (In reply to comment #9) > Is this OK (at config time)? > > checking for NVTVSIMPLE... *** nvtvsimple support will be disabled *** Yes, since there's been no support in rpmfusion either and xine works without it. NVTV is for enabling TV-out on Nvidia cards. nvtv upstream is here: http://sourceforge.net/project/showfiles.php?group_id=33758 nvtvsimple has been updated last in 2004 (nvtv-runtime in 2006), so I don't think it's an important addition. -- 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 501130] New: Review Request: drbdlinks - A program for managing links into a DRBD shared partition
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. Summary: Review Request: drbdlinks - A program for managing links into a DRBD shared partition https://bugzilla.redhat.com/show_bug.cgi?id=501130 Summary: Review Request: drbdlinks - A program for managing links into a DRBD shared partition Product: Fedora Version: rawhide Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: low Component: Package Review AssignedTo: nob...@fedoraproject.org ReportedBy: redhat-bugzi...@linuxnetz.de QAContact: extras...@fedoraproject.org CC: nott...@redhat.com, tcall...@redhat.com, ke...@tummy.com, fedora-package-review@redhat.com, sch...@etes.de Estimated Hours: 0.0 Classification: Fedora Spec URL: http://labs.linuxnetz.de/bugzilla/drbdlinks.spec SRPM URL: http://labs.linuxnetz.de/bugzilla/drbdlinks-1.15-1.src.rpm Description: The drbdlinks program manages links into a DRBD partition which is shared among several machines. A simple configuration file, "/etc/drbdlinks.conf", specifies the links. This can be used to manage e.g. links for /etc/httpd, /var/lib/pgsql and other system directories that need to appear as if they are local to the system when running applications after the drbd shared partition has been mounted. When running drbdlinks with "start" as the mode, drbdlinks will rename the existing files/directories and then make symbolic links into the DRBD partition, "stop" does the reverse. By default, rename appends ".drbdlinks" to the name, but this can be overridden. An init script is included which runs "stop" before heartbeat starts, and after heartbeat stops. This is done to try to ensure that when the shared partition isn't mounted, the links are in their normal state. The drbdlinks upstream package ships a GPLv2 file, but nowhere a licensing is mentioned. According to what I know, Tom is treating such a behaviour normally as GPL+ in the spec file. -- 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 501006] Review Request: xine-ui - A skinned xlib-based gui for xine-lib
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=501006 --- Comment #11 from Zarko (grof) 2009-05-16 16:25:30 EDT --- (In reply to comment #10) > > Stupid, yes. A packaging bug, no. > > > nvtvsimple has been updated last in 2004 (nvtv-runtime in 2006), so I don't > think it's an important addition. OK than. Now it's clear to me, thanks :) -- 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 459455] Review Request: fmit - Free Music Instrument Tuner
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=459455 --- Comment #30 from jebba 2009-05-16 17:26:11 EDT --- Heh. I just got a flood of messages since my email addr just came back online, one of which was about this bug I took a "break" in November 2008 and it got turned into a much longer break when my computers were stolen from my house. Gar. I am back online again, but I am not used Fedora or its derivatives (), so Henrique feel free to take over! Oh, and the files referenced above, if needed, can be had (for a limited time only) at: ftp://216.17.145.32/pub/BLAG/developers/jebba/jebbadora/ Thanks! -- 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 501134] New: Review Request: rubygem-mongrel_cluster - GemPlugin wrapper for the mongrel HTTP server
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. Summary: Review Request: rubygem-mongrel_cluster - GemPlugin wrapper for the mongrel HTTP server https://bugzilla.redhat.com/show_bug.cgi?id=501134 Summary: Review Request: rubygem-mongrel_cluster - GemPlugin wrapper for the mongrel HTTP server Product: Fedora Version: rawhide Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: low Component: Package Review AssignedTo: nob...@fedoraproject.org ReportedBy: redhat-bugzi...@linuxnetz.de QAContact: extras...@fedoraproject.org CC: nott...@redhat.com, fedora-package-review@redhat.com, sch...@etes.de Estimated Hours: 0.0 Classification: Fedora Spec URL: http://labs.linuxnetz.de/bugzilla/rubygem-mongrel_cluster.spec SRPM URL: \ http://labs.linuxnetz.de/bugzilla/rubygem-mongrel_cluster-1.0.5-1.src.rpm Description: Mongrel_cluster is a GemPlugin that wrappers the mongrel HTTP server and simplifies the deployment of webapps using a cluster of mongrel servers. Mongrel_cluster will conveniently configure and control several mongrel servers or groups of mongrel servers, which are then load balanced using a reverse proxy solution. -- 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 480607] Review Request: bluecove - Implementation of JSR-82 Java Bluetooth API
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=480607 --- Comment #4 from François Kooman 2009-05-16 17:42:31 EDT --- Update to latest snapshot, and moved to fedorapeople hosting. Spec URL: http://fkooman.fedorapeople.org/bluecove/bluecove.spec SRPM URL: http://fkooman.fedorapeople.org/bluecove/bluecove-2.1.1-0.20090505.fc11.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 492996] Review Request: mediawiki-Renameuser - An extension that provides a special page for renaming user accounts
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=492996 Jason Tibbitts changed: What|Removed |Added Blocks||182235(FE-Legal) --- Comment #4 from Jason Tibbitts 2009-05-16 19:38:50 EDT --- First off, README.fedora still talks about some module called InputBox which doesn't seem related to RenameUser. Could you fix that up? Now, we can trust a license indicated on an upstream web site as long as we reasonably believe that the authors of the software in question were the ones who made the indication of the license on that site. However, in this case, according to the wiki history it's it's you who've made the indication that it's GPLv2+. So, besides fixing the README.fedora file, I see two options which will allow me to approve this: 1) Spot acks what's being done. I've blocked FE-Legal to request his input. 2) You include in this package the relevant parts of the email conversations with the authors that you indicated you've had on the upstream wiki page. Sorry, licensing sucks. -- 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 474992] Review Request: libirman - Library for IRMAN hardware
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=474992 --- Comment #21 from Jason Tibbitts 2009-05-16 19:47:33 EDT --- OK, I found some time. Unfortunately the package in comment #17 does not build for me in current F11. I get a bunch of bizarre errors with libtool such as: ./libtool: line 793: X--tag=CC: command not found ./libtool: line 826: libtool: ignoring unknown tag : command not found which cascade into: ./libtool: line 1103: X.deps/libirman_la-chunk.Tpo: No such file or directory ./libtool: line 1103: X-c: command not found ./libtool: line 1154: Xlibirman_la-chunk.lo: command not found ./libtool: line 1159: libtool: compile: cannot determine name of library object from `': command not found To rule out problems with my local mock setup, I did a koji build which failed as well: http://koji.fedoraproject.org/koji/taskinfo?taskID=1358328 It does build in F10, though. Can you work out what's broken in F11, or would you like me to review an F10 build? -- 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 446841] Review Request: python-sippy - B2BUA SIP call controlling component
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=446841
--- Comment #11 from Jason Tibbitts 2009-05-16 19:52:35
EDT ---
OK, this does build on F11, rpmlint is clean, the versioning is proper and
%description looks OK to me. However, you still have the conditional
%if 0%{?fedora} >= 8
which will always be satisfied. Is there some reason you can't remove that?
It will not help your epel builds, because %fedora is never defined there.
--
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 491767] Review Request: nss-ldapd - An nsswitch module which uses directory servers
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=491767 --- Comment #24 from Jason Tibbitts 2009-05-16 20:09:03 EDT --- Unfortunately I completely ran out of time to do any reviewing for some time. Getting back to this, I see that the only real issues I had with this past comment #13 were in actually getting it working (which cleared up with selinux turned off). I've been running it on a couple of test servers, and while I haven't really tested it under failure conditions it at least seems to work OK. Can you simply add a note in %description or in a README.Fedora file or something which warns people not to try this with selinux? There's no hard requirement that selinux be supported before a package is imported, though with something that has the potential to lock people out of their machines I really think it's worth making a prominent warning. -- 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 459874] Review Request: zeromq - Fast messaging system
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=459874 --- Comment #21 from Jason Tibbitts 2009-05-16 20:29:03 EDT --- This one fails to build for me in current F11; I get a bunch of errors like: ./zmq/atomic_ptr.hpp:114:6: error: #elif with no expression ./zmq/atomic_ptr.hpp:154:6: error: #elif with no expression in several different files. A failing scratch build is at http://koji.fedoraproject.org/koji/taskinfo?taskID=1358350 An F10 build succeeds. I can review it if you can't figure out what's breaking on F11, but it seems pointless to import a package that won't build on our new release. -- 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 501130] Review Request: drbdlinks - A program for managing links into a DRBD shared partition
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=501130 Robert Scheck changed: What|Removed |Added CC|tcall...@redhat.com | --- Comment #1 from Robert Scheck 2009-05-16 20:51:57 EDT --- Upstream clarified licensing to GPLv2-only with the 1.16 release now: Spec URL: http://labs.linuxnetz.de/bugzilla/drbdlinks.spec SRPM URL: http://labs.linuxnetz.de/bugzilla/drbdlinks-1.16-1.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 498873] Review Request: thrift - A multi-language RPC and serialization framework
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=498873 --- Comment #3 from Silas Sewell 2009-05-16 21:36:16 EDT --- C# bug reported: https://issues.apache.org/jira/browse/THRIFT-509 -- 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 499394] Review Request: ampache - web based audio/video streaming application
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=499394 --- Comment #9 from David Nalley 2009-05-17 00:03:12 EDT --- Several of your install commands don't use -p and thus don't preserve timestamps. Additionally, your cp commands don't consistently use the same parameters, and while technically correct, may be something you want to look out, though it's certainly not a blocker, more of a style issue if anything. Why are you deleting the man page? You just copied the file there just a few lines earlier [ke4...@nalleyt61 SPECS]$ rpmlint ./ampache.spec 0 packages and 1 specfiles checked; 0 errors, 0 warnings. [ke4...@nalleyt61 SPECS]$ rpmlint ../SRPMS/ampache-3.5-8.fc10.src.rpm 1 packages and 0 specfiles checked; 0 errors, 0 warnings. [ke4...@nalleyt61 SPECS]$ rpmlint ../RPMS/noarch/ampache-3.5-8.fc10.noarch.rpm ampache.noarch: E: non-executable-script /usr/share/ampache/locale/base/gather-messages.sh 0644 ampache.noarch: W: non-standard-uid /var/log/ampache apache ampache.noarch: W: non-standard-gid /etc/ampache.cfg apache ampache.noarch: E: non-readable /etc/ampache.cfg 0640 1 packages and 0 specfiles checked; 2 errors, 2 warnings. The modules directory contains a lot of 3rd party packages, such as getid3 from http://getid3.sourceforge.net/ and pear xml rpc from http://pear.php.net/package/XML_RPC There appear to be plenty of these. Unfortunately the packaging guidelines don't permit including other software packages within a single package. Unfortunately this is a common problem with web applications since much of their target audience is shared hosting where the user doesn't have control over the host. If these already exist in fedora you can get either symlink or modify the source to look elsewhere. If they don't you'll need to get those packaged as well and symlink or modify source to point to 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 226429] Merge Review: sqlite
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=226429 --- Comment #16 from Robin Norwood 2009-05-17 00:29:52 EDT --- Sorry, one more: rpmlint points out that the -rpath issue is present here. Ref: https://fedoraproject.org/wiki/Packaging/Guidelines#Beware_of_Rpath The suggested method using sed to strip out the hardcoded paths seems to work for me at first glance. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
