[Bug 215569] Review Request: beryl-vidcap - Beryl OpenGL window and compositing manager video capture utility
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: beryl-vidcap - Beryl OpenGL window and compositing manager video capture utility https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=215569 --- Additional Comments From [EMAIL PROTECTED] 2006-11-17 03:32 EST --- Created an attachment (id=141457) -- (https://bugzilla.redhat.com/bugzilla/attachment.cgi?id=141457action=view) spec file to try to aviod using chrpath Well, using chrpath is somewhat unwilling, I think. How do you think about the spec file I attached? Note: The change for 'make' command in %install process is needed to pass fedora specific compilation flags. NOTE: I only checked about chrpath and $RPM_OPT_FLAGS issues and not checked anything else!! -- 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 215569] Review Request: beryl-vidcap - Beryl OpenGL window and compositing manager video capture utility
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: beryl-vidcap - Beryl OpenGL window and compositing manager video capture utility https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=215569 [EMAIL PROTECTED] changed: What|Removed |Added Attachment #141458|application/octet-stream|text/plain mime type|| -- 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 211319] Review Request: andutteye-client - Andutteye Software Suite Monitoring agent
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: andutteye-client - Andutteye Software Suite Monitoring
agent
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=211319
--- Additional Comments From [EMAIL PROTECTED] 2006-11-17 05:01 EST ---
Many improvements! :-) This is really coming along well.
Your 2 questions:
The agent is running as the normal andutteye user and dont have permission to
create and append to a log file under /var/log. What approach should i use to
over come that?
You can create the log file during %install (using touch or something similar),
and change its ownership to the andutteye user, or you can create a andutteye
or andutteyed subdir in /var/log, with ownership set to the andutteyed user;
in this case you can simply own the directory itself in %files.
You also need to create a entry in /etc/logrotate.d to control the size of the
log file(s); this can also be used to set ownership/permissions, etc (look at
existing configurations in /etc/logrotate.d for examples; yum has a simple but
effective entry, ppp as well).
The andutteye user should have /sbin in its PATH so that all statistics can be
retrived. This is only nessasary if the agent is connected to a AES server
though. What is the best approach for this?
The user account that gets created should actually be a system account, i.e. not
one that a normal user can be used to log in as. You can use something like:
%pre
# Create system account
/usr/sbin/useradd -g andutteye \
-c Andutteye Software Suite monitoring server -r -M -s '' \
-d %{_localstatedir}/cache/andutteye andutteye /dev/null || :
Note the -r flag there (for creating a system account), and setting the home
directory to /var/cache/andutteye. Since this service is then started as a
system account by init, it *should* include /sbin/ in its PATH... Let me know
how it goes - I'm very rusty on this, so if anyone knows better, please advise.
Some other stuff:
* %doc entries: permissions are incorrect (0644, should be 0755)
- by the way, you can (if you like) place all the doc entries in one %doc
statement, by seperating them with spaces
* Man pages are installed in the wrong location (they are not accessible).
Suggested fix:
%install
[...snip...]
mkdir -p %{buildroot}%{_mandir}/man1
install -p -m 0755 usr/share/man/man1/* %{buildroot}%{_mandir}/man1
[...snip...]
%files
[...snip...]
%{_mandir}/man1/*
[...snip...]
* I've been delaying mentioning this, but your Source statement needs to have
some sort of information on HOW to obtain the source tarball. Usually this is
done by specifying the full URL where the source file(s) can be downloaded,
i.e.
Source0:
http://www.andutteye.com/download/andutteye-client-%{version}-%{release}.tar.gz
There is a slight problem here, though. I've noticed that you increment a
release tag of the source tarball for each successive RPM build (to
synchronize the two releases). This is your choice, of course (you're the
author, after all), but does anything actually change in the tar.gz release? It
would be easier to maintain this package if you leave the release tag to
indicate *RPM* releases, and just keep a version tag for the tarball (this is
how it usually works, anyhow).
So, in short, how it usually works:
RPM packages foo-1.3-1, foo-1.3-5, foo-1.3-6 all use the same source:
foo-1.3.tar.gz
The different release tags merely state that something specific to the RPM has
changed, not the upstream program. In this fashion, if the source tarball
changes (say, to foo-1.3.1.tar.gz), the RPM's release tag is reset, so the RPM
package becomes foo-1.3.1-1.tar.gz.
Notice that the release tag only shows changes in the RPM itself (specfile
changes, etc), not changes in the upstream source. And everytime the source
changes, RPM's release tag goes back to 1.
This is a packaging rule in Fedora, so if you decide to keep the -release in
the upstream tarball, you have to put that into your Version tag in the spec,
and change the - character to something else, perhaps a '.'. In other words,
for the tarball andutteye-client-2.3-7.tar.gz (on the website), the resulting
RPM package should be something like:
andutteye-client-2.3.7-1.noarch.rpm
or
andutteye-client-2.3_7-1.noarch.rpm
* rpmlint isn't silent (run it on the binary rpm, not the .src.rpm :-) ), but a
lot of the output has to do with the andutteye user account, as well as the
man-page stuff. Some interesting entries that need to be addressed:
E: andutteye-client non-readable /etc/andutteye/andutteyed.conf 0640
W: andutteye-client non-conffile-in-etc /etc/andutteye/andutteyed.conf
-- change the permissions for this file, and mark it as %conf in %files
W: andutteye-client service-default-enabled /etc/init.d/andutteyed
-- I don't have enough experience to comment on this. Depending on the service,
it may or may not be a good thing to have this specific
[Bug 208169] Review Request: python-twisted-core - An asynchronous networking framework written in Python
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: python-twisted-core - An asynchronous networking framework written in Python https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=208169 --- Additional Comments From [EMAIL PROTECTED] 2006-11-17 05:05 EST --- so, poll time. Should I already request builds for FC6 and devel, or should we first move on with the rest of the twisted stack until we have a feature-complete replacement for the FC-5 complete python-twisted package ? Creating tickets for the next packages -- 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 216102] New: Review Request: python-twisted-conch - Twisted SSHv2 implementation
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=216102 Summary: Review Request: python-twisted-conch - Twisted SSHv2 implementation Product: Fedora Extras Version: devel Platform: All OS/Version: Linux Status: NEW Severity: normal Priority: normal Component: Package Review AssignedTo: [EMAIL PROTECTED] ReportedBy: [EMAIL PROTECTED] QAContact: fedora-package-review@redhat.com Spec URL: http://thomas.apestaart.org/download/pkg/fedora-5-x86_64-extras/python-twisted-conch-0.7.0-3.fc5/python-twisted-conch.spec SRPM URL: http://thomas.apestaart.org/download/pkg/fedora-5-x86_64-extras/python-twisted-conch-0.7.0-3.fc5/python-twisted-conch-0.7.0-3.fc5.src.rpm Description: Twisted is an event-based framework for internet applications. Conch is an SSHv2 implementation written in Python. SSH is a protocol designed to allow remote access to shells and commands, but it is generic enough to allow everything from TCP forwarding to generic filesystem access. Since conch is written in Python, it interfaces well with other Python projects, such as Imagination. Conch also includes a implementations of the telnet and vt102 protocols, as well as support for rudamentary line editing behaviors. A new implementation of Twisted's Manhole application is also included, featuring server-side input history and interactive syntax coloring. -- 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 216103] New: Review Request: twisted-names - A Twisted DNS implementation
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=216103 Summary: Review Request: twisted-names - A Twisted DNS implementation Product: Fedora Extras Version: devel Platform: All OS/Version: Linux Status: NEW Severity: normal Priority: normal Component: Package Review AssignedTo: [EMAIL PROTECTED] ReportedBy: [EMAIL PROTECTED] QAContact: fedora-package-review@redhat.com Spec URL: http://thomas.apestaart.org/download/pkg/fedora-5-x86_64-extras/python-twisted-names-0.3.0-2.fc5/python-twisted-names.spec SRPM URL: http://thomas.apestaart.org/download/pkg/fedora-5-x86_64-extras/python-twisted-names-0.3.0-2.fc5/python-twisted-names-0.3.0-2.fc5.src.rpm Description: Twisted is an event-based framework for internet applications. Twisted Names is both a domain name server as well as a client resolver library. Twisted Names comes with an out of the box nameserver which can read most BIND-syntax zone files as well as a simple Python-based configuration format. Twisted Names can act as an authoritative server, perform zone transfers from a master to act as a secondary, act as a caching nameserver, or any combination of these. Twisted Names' client resolver library provides functions to query for all commonly used record types as well as a replacement for the blocking gethostbyname() function provided by the Python stdlib socket module. -- 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 216104] New: Review Request: twisted-runner - process management library and inetd replacement
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=216104 Summary: Review Request: twisted-runner - process management library and inetd replacement Product: Fedora Extras Version: devel Platform: All OS/Version: Linux Status: NEW Severity: normal Priority: normal Component: Package Review AssignedTo: [EMAIL PROTECTED] ReportedBy: [EMAIL PROTECTED] QAContact: fedora-package-review@redhat.com Spec URL: http://thomas.apestaart.org/download/pkg/fedora-5-x86_64-extras/python-twisted-runner-0.2.0-3.fc5/python-twisted-runner.spec SRPM URL: http://thomas.apestaart.org/download/pkg/fedora-5-x86_64-extras/python-twisted-runner-0.2.0-3.fc5/python-twisted-runner-0.2.0-3.fc5.src.rpm Description: Twisted is an event-based framework for internet applications. Twisted Runner contains code useful for persistent process management with Python and Twisted, and has an almost full replacement for inetd. -- 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 216105] New: Review Request: python-twisted-web - Twisted web server, programmable in Python
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=216105 Summary: Review Request: python-twisted-web - Twisted web server, programmable in Python Product: Fedora Extras Version: devel Platform: All OS/Version: Linux Status: NEW Severity: normal Priority: normal Component: Package Review AssignedTo: [EMAIL PROTECTED] ReportedBy: [EMAIL PROTECTED] QAContact: fedora-package-review@redhat.com Spec URL: http://thomas.apestaart.org/download/pkg/fedora-5-x86_64-extras/python-twisted-web-0.6.0-3.fc5/python-twisted-web.spec SRPM URL: http://thomas.apestaart.org/download/pkg/fedora-5-x86_64-extras/python-twisted-web-0.6.0-3.fc5/python-twisted-web-0.6.0-3.fc5.src.rpm Description: Twisted is an event-based framework for internet applications. Twisted Web is a complete web server, aimed at hosting web applications using Twisted and Python, but fully able to serve static pages, also. -- 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 216106] New: Review Request: python-twisted-words - Twisted Words contains Instant Messaging implementations
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=216106 Summary: Review Request: python-twisted-words - Twisted Words contains Instant Messaging implementations Product: Fedora Extras Version: devel Platform: All OS/Version: Linux Status: NEW Severity: normal Priority: normal Component: Package Review AssignedTo: [EMAIL PROTECTED] ReportedBy: [EMAIL PROTECTED] QAContact: fedora-package-review@redhat.com Spec URL: http://thomas.apestaart.org/download/pkg/fedora-5-x86_64-extras/python-twisted-words-0.4.0-2.fc5/python-twisted-words.spec SRPM URL: http://thomas.apestaart.org/download/pkg/fedora-5-x86_64-extras/python-twisted-words-0.4.0-2.fc5/python-twisted-words-0.4.0-2.fc5.src.rpm Description: Twisted is an event-based framework for internet applications. Twisted Words contains implementations of many Instant Messaging protocols, including IRC, Jabber, MSN, OSCAR (AIM ICQ), TOC (AOL), and some functionality for creating bots, inter-protocol gateways, and a client application for many of the protocols. In support of Jabber, Twisted Words also contains X-ish, a library for processing XML with Twisted and Python, with support for a Pythonic DOM and an XPath-like toolkit. -- 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 216103] Review Request: python-twisted-names - A Twisted DNS implementation
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: python-twisted-names - A Twisted DNS implementation https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=216103 [EMAIL PROTECTED] changed: What|Removed |Added Summary|Review Request: twisted-|Review Request: python- |names - A Twisted DNS |twisted-names - A Twisted |implementation |DNS implementation -- 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 216104] Review Request: python-twisted-runner - process management library and inetd replacement
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: python-twisted-runner - process management library and inetd replacement https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=216104 [EMAIL PROTECTED] changed: What|Removed |Added Summary|Review Request: twisted-|Review Request: python- |runner - process management |twisted-runner - process |library and inetd |management library and inetd |replacement |replacement -- 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 215165] Review Request: audacious-plugins - Plugins for the Audacious media player
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: audacious-plugins - Plugins for the Audacious media player https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=215165 [EMAIL PROTECTED] changed: What|Removed |Added CC||[EMAIL PROTECTED] --- Additional Comments From [EMAIL PROTECTED] 2006-11-17 05:20 EST --- Yeah, this also seems correct to me : No need to provide the old names, as they shouldn't have been used other than explicitly (not from other package requirements). The obsoletes are required though, to provide a clean upgrade path, and are correct with the last known version. Please start the review ASAP or let me know if you'd like me to do it instead, as I'm quite impatient to have this audacious update available ;-) -- 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 208169] Review Request: python-twisted-core - An asynchronous networking framework written in Python
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: python-twisted-core - An asynchronous networking framework written in Python https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=208169 --- Additional Comments From [EMAIL PROTECTED] 2006-11-17 05:47 EST --- I think we should review, import, and build all of the packages in a bottom-up dependency order. So the python-twisted metapackage would be the last one, and that is the only one that has been present in previous releases, so that's the only one that should cause an issue for anyone, so long as nobody builds any other package requiring one of the new python-twisted-* packages before the top-level metapackage is available. -- 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 215165] Review Request: audacious-plugins - Plugins for the Audacious media player
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: audacious-plugins - Plugins for the Audacious media player https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=215165 --- Additional Comments From [EMAIL PROTECTED] 2006-11-17 06:09 EST --- Ok. Based on you comment i am going to review this package now. -- 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 215165] Review Request: audacious-plugins - Plugins for the Audacious media player
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: audacious-plugins - Plugins for the Audacious media player https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=215165 --- Additional Comments From [EMAIL PROTECTED] 2006-11-17 06:41 EST --- (In reply to comment #7) Yeah, this also seems correct to me : No need to provide the old names, as they shouldn't have been used other than explicitly (not from other package requirements). The obsoletes are required though, to provide a clean upgrade path, and are correct with the last known version. Please start the review ASAP or let me know if you'd like me to do it instead, as I'm quite impatient to have this audacious update available ;-) Thanks for testing this package. Based on above comment Review + package builds in mock (development i386)FC7. + rpmlint is silent for SRPM. + rpmlint on RPMs is not silent but as per above comments in bugzilla they can be ignored. + source files match upstream. 8ac7f73da7432e1ffed6c2b9b0fced8c audacious-plugins-fedora-1.2.2.tar.gz + package meets naming and packaging guidelines. + specfile is properly named, is cleanly written + Spec file is written in American English. + Spec file is legible. + dist tag is present. + build root is correct. + license is open source-compatible. License text included in package. + %doc is small; no -doc subpackage required. + %doc does not affect runtime. + COPYING included in %doc. + BuildRequires are proper. + %clean is present. + package installed properly. + Macro use appears rather consistent. + Package contains code, not content. + no headers or static libraries. + no .pc files. + no -devel subpackage exists + available subpackages are audacious-plugins-jack,audacious-plugins-arts, audacious-plugins-esd,audacious-plugins-pulseaudio + as subpackages are packaging .so files post and postun called /sbin/ldconfig + Used update-desktop-database correctly + no .la files. + no translations available - Does NOT owns the directories it creates. + no duplicates in %files. + file permissions are appropriate. SHOULD:- I saw that some directories are owned in audacious rpm whereas there is no need to own them by audacious but it should be owned by audacious-plugins. like /usr/lib/audacious /usr/lib/audacious/Container /usr/lib/audacious/Effect /usr/lib/audacious/General /usr/lib/audacious/Input /usr/lib/audacious/Output /usr/lib/audacious/Visualization Make it own by audacious-plugins -- 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 195645] Review Request: rasqal - RDF query library
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: rasqal - RDF query library https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=195645 --- Additional Comments From [EMAIL PROTECTED] 2006-11-17 06:41 EST --- http://thomas.apestaart.org/download/pkg/fedora-5-x86_64-extras/rasqal-0.9.12-3.fc5/ updated -- 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 215165] Review Request: audacious-plugins - Plugins for the Audacious media player
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: audacious-plugins - Plugins for the Audacious media player https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=215165 --- Additional Comments From [EMAIL PROTECTED] 2006-11-17 06:56 EST --- Matthias Saou, What do you think is above question is valid or should i approve package as it is? -- 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 209260] Review Request: beryl-manager - Beryl window decorator and theme management utility
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: beryl-manager - Beryl window decorator and theme management utility https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=209260 [EMAIL PROTECTED] changed: What|Removed |Added CC||[EMAIL PROTECTED] --- Additional Comments From [EMAIL PROTECTED] 2006-11-17 06:57 EST --- Are the names of these po/mo files correct? ar_AR, tr_TR, my_MY etc. They are creating new directories under /usr/share/locale that no package own. -- 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 215165] Review Request: audacious-plugins - Plugins for the Audacious media player
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: audacious-plugins - Plugins for the Audacious media player https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=215165 --- Additional Comments From [EMAIL PROTECTED] 2006-11-17 07:07 EST --- OK, one might argue that a-p should own /usr/lib/audacious, since audacious itself does not place any files there. If (and that's a big if) we/I should ever decide to split up a-p into a lot of tiny subpackages these directories would have to go back to audacious, since no subpackage could own all of these dirs. I personally do not care too much about it, but given the choice would leave things as they are (reduces work for me :) -- 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 215165] Review Request: audacious-plugins - Plugins for the Audacious media player
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: audacious-plugins - Plugins for the Audacious media player https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=215165 --- Additional Comments From [EMAIL PROTECTED] 2006-11-17 07:25 EST --- Since we have audacious requiring audacious-plugins, and audacious-plugins requiring audacious, I'm not quite sure what the proper solution would be. A while back, I would have made those dirs owned by both packages, in order to be 100% sure they get rmdir'ed upon removal of both packages (since we don't know which will be removed last in the same rpm transaction), but I'm not sure this is compatible with today's packaging guidelines. You might want to ask on the packaging list or go through the guidelines again ;-) -- 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 215165] Review Request: audacious-plugins - Plugins for the Audacious media player
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: audacious-plugins - Plugins for the Audacious media player https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=215165 --- Additional Comments From [EMAIL PROTECTED] 2006-11-17 07:31 EST --- Erm, audacious Requires: audacious-plugins. audacious-plugins Requires: libaudacious.so.4 (provided by audacious-libs) audacious-plugins does not require audacious (circular deps, ugly). So there is nothing that prevents audacious from being removed while a-p is still installed as far as I can see. Ugly, that. Make audacious-libs own the plugin-dirs? -- 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 214669] Review Request: logserial
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: logserial https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=214669 [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 215165] Review Request: audacious-plugins - Plugins for the Audacious media player
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: audacious-plugins - Plugins for the Audacious media player https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=215165 --- Additional Comments From [EMAIL PROTECTED] 2006-11-17 08:16 EST --- I just looked again at audacious.spec : Requires: audacious-plugins = 1.2.0 And at audacious-plugins.spec : Requires: audacious = 1.2.0 But maybe those two spec files aren't the latest? -- 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 216129] New: Review Request: xerces-c - Validating XML Parser
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=216129 Summary: Review Request: xerces-c - Validating XML Parser Product: Fedora Extras Version: devel Platform: All OS/Version: Linux Status: NEW Severity: normal Priority: normal Component: Package Review AssignedTo: [EMAIL PROTECTED] ReportedBy: [EMAIL PROTECTED] QAContact: fedora-package-review@redhat.com Spec URL: http://gauret.free.fr/fichiers/rpms/fedora/xerces-c.spec SRPM URL: http://gauret.free.fr/fichiers/rpms/fedora/xerces-c-2.7.0-1.src.rpm Description: Xerces-C is a validating XML parser written in a portable subset of C++. Xerces-C makes it easy to give your application the ability to read and write XML data. A shared library is provided for parsing, generating, manipulating, and validating XML documents. Xerces-C is faithful to the XML 1.0 recommendation and associated standards ( DOM 1.0, DOM 2.0. SAX 1.0, SAX 2.0, Namespaces). This package has been imported from Dag's repo and slightly tweaked. Thanks to Dag for his work. -- 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 216130] New: Review Request: glest - 3D real time strategy game
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=216130 Summary: Review Request: glest - 3D real time strategy game Product: Fedora Extras Version: devel Platform: All OS/Version: Linux Status: NEW Severity: normal Priority: normal Component: Package Review AssignedTo: [EMAIL PROTECTED] ReportedBy: [EMAIL PROTECTED] QAContact: fedora-package-review@redhat.com Spec URL: http://gauret.free.fr/fichiers/rpms/fedora/glest.spec SRPM URL: http://gauret.free.fr/fichiers/rpms/fedora/glest-2.0.0-1.src.rpm Description: Glest is a free 3D real time strategy game, that can be modified using XML and a set of tools Project's website : http://www.glest.org -- 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 216131] New: Review Request: glest-data - Data files for the game Glest
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=216131 Summary: Review Request: glest-data - Data files for the game Glest Product: Fedora Extras Version: devel Platform: All OS/Version: Linux Status: NEW Severity: normal Priority: normal Component: Package Review AssignedTo: [EMAIL PROTECTED] ReportedBy: [EMAIL PROTECTED] QAContact: fedora-package-review@redhat.com Spec URL: http://gauret.free.fr/fichiers/rpms/fedora/glest-data.spec SRPM URL: http://gauret.free.fr/fichiers/rpms/fedora/glest-data-2.0.0-1.src.rpm Description: Glest is a free 3D real time strategy game, that can be modified using XML and a set of tools This package contains the data files for the game See bug 216130 for the game engine. -- 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 216130] Review Request: glest - 3D real time strategy game
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: glest - 3D real time strategy game https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=216130 [EMAIL PROTECTED] changed: What|Removed |Added BugsThisDependsOn||216129, 216131 -- 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 216129] Review Request: xerces-c - Validating XML Parser
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: xerces-c - Validating XML Parser https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=216129 [EMAIL PROTECTED] changed: What|Removed |Added OtherBugsDependingO||216130 nThis|| -- 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 216131] Review Request: glest-data - Data files for the game Glest
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: glest-data - Data files for the game Glest https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=216131 [EMAIL PROTECTED] changed: What|Removed |Added OtherBugsDependingO||216130 nThis|| -- 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 215165] Review Request: audacious-plugins - Plugins for the Audacious media player
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: audacious-plugins - Plugins for the Audacious media player https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=215165 --- Additional Comments From [EMAIL PROTECTED] 2006-11-17 09:05 EST --- You're right, I forgot that (splitting off audacious-libs solved the build time deps, however). But even the explicit dependency does not guarantee removal in the right order, or does it? Any bright ideas on that? -- 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 214730] Review Request: pungi - Distribution compose tool
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: pungi - Distribution compose tool https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=214730 --- Additional Comments From [EMAIL PROTECTED] 2006-11-17 09:11 EST --- Shouldn't this bug be closed now? -- 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 214830] Review Request: Limph - PHP network host monitor
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: Limph - PHP network host monitor https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=214830 --- Additional Comments From [EMAIL PROTECTED] 2006-11-17 09:21 EST --- Spec URL: http://gryffindor.jcomserv.net/extras/limph/limph-2.spec SRPM URL: http://gryffindor.jcomserv.net/extras/limph/limph-1.9.3-2.src.rpm I addressed the source BuildArch, URL, RPM_SOURCE_DIR, limph.conf issues. I'm trying to solve the conf file issues with symlinks and it's not working, nor is my %attr for the tmp directory applying the desired ownership. What am I doing wrong? The reason the MD5 doesn't match upstream is the folder name in the source tarball is limph, not limph-1.9.3, so I untarred, renamed and retarred to get it to build. Should I re-release a 1.9.3.1 with this fixed, or is there a way around it in the SPEC? Thanks for the help, BTW. -- 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 210553] Review Request: xerces-c - Validating XML Parser
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: xerces-c - Validating XML Parser https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=210553 [EMAIL PROTECTED] changed: What|Removed |Added CC||[EMAIL PROTECTED] --- Additional Comments From [EMAIL PROTECTED] 2006-11-17 09:28 EST --- *** Bug 216129 has been marked as a duplicate of this bug. *** -- 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 214730] Review Request: pungi - Distribution compose tool
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: pungi - Distribution compose tool https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=214730 [EMAIL PROTECTED] changed: What|Removed |Added Status|ASSIGNED|CLOSED Resolution||RAWHIDE --- Additional Comments From [EMAIL PROTECTED] 2006-11-17 09:28 EST --- Whoops, I was waiting for the push, that happened. -- 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 216130] Review Request: glest - 3D real time strategy game
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: glest - 3D real time strategy game https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=216130 Bug 216130 depends on bug 216129, which changed state. Bug 216129 Summary: Review Request: xerces-c - Validating XML Parser https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=216129 What|Old Value |New Value Resolution||DUPLICATE Status|NEW |CLOSED -- 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 215568] Review Request: beryl-dbus - Beryl OpenGL window and compositing manager dbus plug-in
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: beryl-dbus - Beryl OpenGL window and compositing manager dbus plug-in https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=215568 [EMAIL PROTECTED] changed: What|Removed |Added OtherBugsDependingO|163778 |163779 nThis|| --- Additional Comments From [EMAIL PROTECTED] 2006-11-17 09:52 EST --- Okay. --- This package (beryl-dbus) is APPROVED by me. --- Two comments. 1. Requires: dbus This is not necessary because this package (beryl-dbus) requires libdbus-1.so.3 and this adds dbus dependency automatically. 2. For beryl-core package: /usr/lib/libberylsettings.so.0.0.0 contains lots of undefined non-weak symbols (as said above). [EMAIL PROTECTED] ~]$ ldd -r /usr/lib/libberylsettings.so.0.0.0 undefined symbol: g_free(/usr/lib/libberylsettings.so.0.0.0) undefined symbol: g_free(/usr/lib/libberylsettings.so.0.0.0) undefined symbol: g_slist_remove(/usr/lib/libberylsettings.so.0.0.0) undefined symbol: g_mkdir_with_parents (/usr/lib/libberylsettings.so.0.0.0) undefined symbol: g_key_file_has_key(/usr/lib/libberylsettings.so.0.0.0) undefined symbol: g_key_file_get_string_list (/usr/lib/libberylsettings.so.0.0.0) undefined symbol: g_dir_close (/usr/lib/libberylsettings.so.0.0.0) undefined symbol: XStringToKeysym (/usr/lib/libberylsettings.so.0.0.0) ... ... Please fix the linkage against this library. -- 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 208169] Review Request: python-twisted-core - An asynchronous networking framework written in Python
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: python-twisted-core - An asynchronous networking framework written in Python https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=208169 --- Additional Comments From [EMAIL PROTECTED] 2006-11-17 10:02 EST --- Not having python-twisted-core pushed out (at least to devel) will make testing builds with mock harder, as you'll have to set up a local repo with python-twisted-core in it and configure mock to look at it. -- 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 215571] Review Request: heliodor - Beryl window decorator with Metacity theme support
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: heliodor - Beryl window decorator with Metacity theme support https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=215571 [EMAIL PROTECTED] changed: What|Removed |Added Status|NEW |ASSIGNED AssignedTo|[EMAIL PROTECTED]|[EMAIL PROTECTED] OtherBugsDependingO|163776 |163778 nThis|| -- 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 215792] Review Request: wgrib - Manipulate, inventory and decode GRIB 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: wgrib - Manipulate, inventory and decode GRIB files https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=215792 --- Additional Comments From [EMAIL PROTECTED] 2006-11-17 11:43 EST --- Emailed Wesley. 1.8.0.12g has just been made production, so I've upped the version. He still wants the .c file to be the official release, but has started versioning them on the ftp site. I've asked again for versioned .tar files so we can get the Changes file with them, but I really don't see it as a blocker at this point. http://www.cora.nwra.com/~orion/fedora/wgrib-1.8.0.12g-1.fc6.src.rpm -- 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 210553] Review Request: xerces-c - Validating XML Parser
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: xerces-c - Validating XML Parser https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=210553 --- Additional Comments From [EMAIL PROTECTED] 2006-11-17 11:48 EST --- Needs work: * Use of buildroot is not consistant (wiki: PackagingGuidelines#UsingBuildRootOptFlags) * Missing SMP flags. If it doesn't build with it, please add a comment (wiki: PackagingGuidelines#parallelmake) * rpmlint: - strange-permission xerces-c.spec 0666 (in the src.rpm) - hidden dir : /usr/share/doc/xerces-c-doc-2.7.0/doc/html/resources/.svn (please clean it up in %%prep) For what it's worth, I'd just move the samples to the doc package -- 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 214024] Review Request: perl-Net-DBus - Net::DBus Perl module
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: perl-Net-DBus - Net::DBus Perl module https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=214024 --- Additional Comments From [EMAIL PROTECTED] 2006-11-17 11:56 EST --- Ok, I found a FC5 box and build it there... with the additional BR's, no problem. I also built it under mock for fc56 (i386)... All tests passed cleanly, so I'm not quite sure what happened above. SRPM URL: http://home.comcast.net/~ckweyl/perl-Net-DBus-0.33.4-2.fc6.src.rpm SPEC URL: http://home.comcast.net/~ckweyl/perl-Net-DBus.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 215569] Review Request: beryl-vidcap - Beryl OpenGL window and compositing manager video capture utility
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: beryl-vidcap - Beryl OpenGL window and compositing
manager video capture utility
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=215569
--- Additional Comments From [EMAIL PROTECTED] 2006-11-17 11:56 EST ---
Well, during review on beryl-vipcap, several issues are
found.
* %{_libdir}/beryl/libcapture.so contains undefined
non-weak symbols.
[EMAIL PROTECTED] ~]$ ldd -r /usr/lib/beryl/libcapture.so /dev/null
undefined symbol: glEnd (/usr/lib/beryl/libcapture.so)
undefined symbol: glEnable (/usr/lib/beryl/libcapture.so)
undefined symbol: glColor4us(/usr/lib/beryl/libcapture.so)
undefined symbol: glEnableClientState (/usr/lib/beryl/libcapture.so)
undefined symbol: glDisable (/usr/lib/beryl/libcapture.so)
undefined symbol: glRecti (/usr/lib/beryl/libcapture.so)
undefined symbol: addScreenAction (/usr/lib/beryl/libcapture.so)
undefined symbol: compSetFloatOption(/usr/lib/beryl/libcapture.so)
undefined symbol: getIntOptionNamed (/usr/lib/beryl/libcapture.so)
(and others)
Please check the linkage against this package.
* Related to above, -devel package are missing necessary
Requires.
%{_includedir}/seom/seom.h reads:
#include GL/gl.h
#include GL/glext.h
#include GL/glx.h
#include X11/Xlib.h
#include X11/Xatom.h
#include X11/keysym.h
This means -devel package requires at least the following packages
---
libGL-devel (this is provided by mesa-libGL-devel)
libX11-devel
xorg-x11-proto-devel (this is required by mesa-libGL-devel so
redundant for writing to Requires)
---
and also this implies that the missing linkage on
%{_libdir}/beryl/libcapture.so _may_ be for
libGL.so and libX11.so (I have not checked).
* Fedora specific compilation flags are not passed.
+ umask 022
+ cd /builddir/build/BUILD
+ cd beryl-vidcap-0.1.2
+ LANG=C
+ export LANG
+ unset DISPLAY
+ rm -rf /var/tmp/beryl-vidcap-0.1.2-6.fc7-root-mockbuild
+ pushd seom
~/build/BUILD/beryl-vidcap-0.1.2/seom ~/build/BUILD/beryl-vidcap-0.1.2
+ make DESTDIR=/var/tmp/beryl-vidcap-0.1.2-6.fc7-root-mockbuild install
cc -W -Wall -std=c99 -Iinclude -ldl -lpthread -L.libs -lseom -o seom-filter
src/filter/main.c
cc -W -Wall -std=c99 -Iinclude -ldl -lpthread -L.libs -lseom -lX11 -lXv -o
seom-player src/player/main.c
cc -W -Wall -std=c99 -Iinclude -ldl -lpthread -L.libs -lseom -o seom-server
src/server/main.c
--
(This uses original 0.1.2-6 spec file). Please fix this.
* License:
I cannot find any GPL license document file in the source
files. Only I can read is in capture.c, however, it is not
GPL (looks like free license, however, what is this?)
Well, other files than capture.c don't contain any license
terms and no license document is provided, so, the license
of this package is really questionable...
--
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 214024] Review Request: perl-Net-DBus - Net::DBus Perl module
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: perl-Net-DBus - Net::DBus Perl module https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=214024 --- Additional Comments From [EMAIL PROTECTED] 2006-11-17 12:21 EST --- The point was that it fails to build in rawhide. I tested it on FC5 and it was fine; I was unable to test on FC6 but I am not surprised to see that it works. Failure to work on rawhide is certainly not a blocker, but you should know up front that the code is buggy and could fail at pretty much any time if Red Hat see a need to update DBUS in FC6. I'll check out the new packages. -- 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 214024] Review Request: perl-Net-DBus - Net::DBus Perl module
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: perl-Net-DBus - Net::DBus Perl module https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=214024 --- Additional Comments From [EMAIL PROTECTED] 2006-11-17 12:32 EST --- Ahh, rawhide. I missed that, somehow. The module appears to be actively maintained, and the author has been fairly responsive when I've filed issues against it. I'm not worried about there being dbus support in this module to match rawhile by the time fc7 rolls around. -- 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 215563] Review Request: aquamarine - Themeable window decorator and compositing manager for Beryl
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: aquamarine - Themeable window decorator and compositing manager for Beryl https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=215563 --- Additional Comments From [EMAIL PROTECTED] 2006-11-17 12:45 EST --- mock build suceeds on FC-devel i386. * BuildRequires: qt, kdelibs This is not needed to be written as libraries' dependency on this package automatically requires them (libqt-mt.so.3 and libkdecore.so.4) -- 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 189188] Review Request: sqlgrey - postfix grey-listing policy service
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: sqlgrey - postfix grey-listing policy service https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=189188 --- Additional Comments From [EMAIL PROTECTED] 2006-11-17 13:08 EST --- I figured out my issue. As-is, the package doesn't depend on any DBD modules, but it defaults to using DBD::Pg: Nov 17 12:01:32 localhost sqlgrey: fatal: Can't locate DBD/Pg.pm in @INC (@INC contains: /usr/lib/perl5/site_perl/5.8.8/i386-linux-thread-multi /usr/lib/perl5/site_perl/5.8.7/i386-linux-thread-multi /usr/lib/perl5/site_perl/5.8.6/i386-linux-thread-multi /usr/lib/perl5/site_perl/5.8.5/i386-linux-thread-multi /usr/lib/perl5/site_perl/5.8.8 /usr/lib/perl5/site_perl/5.8.7 /usr/lib/perl5/site_perl/5.8.6 /usr/lib/perl5/site_perl/5.8.5 /usr/lib/perl5/site_perl /usr/lib/perl5/vendor_perl/5.8.8/i386-linux-thread-multi /usr/lib/perl5/vendor_perl/5.8.7/i386-linux-thread-multi /usr/lib/perl5/vendor_perl/5.8.6/i386-linux-thread-multi /usr/lib/perl5/vendor_perl/5.8.5/i386-linux-thread-multi /usr/lib/perl5/vendor_perl/5.8.8 /usr/lib/perl5/vendor_perl/5.8.7 /usr/lib/perl5/vendor_perl/5.8.6 /usr/lib/perl5/vendor_perl/5.8.5 /usr/lib/perl5/vendor_perl /usr/lib/perl5/5.8.8/i386-linux-thread-multi /usr/lib/perl5/5.8.8 .) at (eval 9) line 3. Nov 17 12:01:33 localhost sqlgrey: fatal: install_driver(Pg) failed: Can't locate DBD/Pg.pm in @INC (@INC contains: /usr/lib/perl5/site_perl/5.8.8/i386-linux-thread-multi /usr/lib/perl5/site_perl/5.8.7/i386-linux-thread-multi /usr/lib/perl5/site_perl/5.8.6/i386-linux-thread-multi /usr/lib/perl5/site_perl/5.8.5/i386-linux-thread-multi /usr/lib/perl5/site_perl/5.8.8 /usr/lib/perl5/site_perl/5.8.7 /usr/lib/perl5/site_perl/5.8.6 /usr/lib/perl5/site_perl/5.8.5 /usr/lib/perl5/site_perl /usr/lib/perl5/vendor_perl/5.8.8/i386-linux-thread-multi /usr/lib/perl5/vendor_perl/5.8.7/i386-linux-thread-multi /usr/lib/perl5/vendor_perl/5.8.6/i386-linux-thread-multi /usr/lib/perl5/vendor_perl/5.8.5/i386-linux-thread-multi /usr/lib/perl5/vendor_perl/5.8.8 /usr/lib/perl5/vendor_perl/5.8.7 /usr/lib/perl5/vendor_perl/5.8.6 /usr/lib/perl5/vendor_perl/5.8.5 /usr/lib/perl5/vendor_perl /usr/lib/perl5/5.8.8/i386-linux-thread-multi /usr/lib/perl5/5.8.8 .) at (eval 9) line 3. Perhaps the DBD::Pg perl module hasn't been fully installed, or per If there are no objections, I'll change the packaged config file to use SQLite (and require DBD::SQLite). -- 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 215568] Review Request: beryl-dbus - Beryl OpenGL window and compositing manager dbus plug-in
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: beryl-dbus - Beryl OpenGL window and compositing manager dbus plug-in https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=215568 --- Additional Comments From [EMAIL PROTECTED] 2006-11-17 13:13 EST --- (In reply to comment #7) Okay. --- This package (beryl-dbus) is APPROVED by me. --- Two comments. 1. Requires: dbus This is not necessary because this package (beryl-dbus) requires libdbus-1.so.3 and this adds dbus dependency automatically. Done for the version I'll import once cvs is resurrected. Thank you! 2. For beryl-core package: /usr/lib/libberylsettings.so.0.0.0 contains lots of undefined non-weak symbols (as said above). [...] ... Please fix the linkage against this library. Looking into it now... -- 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 215563] Review Request: aquamarine - Themeable window decorator and compositing manager for Beryl
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: aquamarine - Themeable window decorator and compositing manager for Beryl https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=215563 --- Additional Comments From [EMAIL PROTECTED] 2006-11-17 13:52 EST --- New build, removes Requires: on qt and kdelibs, relies on auto-gen'd lib deps. http://wilsonet.com/packages/beryl/aquamarine-0.1.2-3.fc6.src.rpm -- 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 215571] Review Request: heliodor - Beryl window decorator with Metacity theme support
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: heliodor - Beryl window decorator with Metacity theme support https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=215571 --- Additional Comments From [EMAIL PROTECTED] 2006-11-17 14:37 EST --- Okay, this is due to changes in metacity 2.17 in rawhide. The *_corner_rounded bits were renamed to *_corner_rounded_radius. Easy enough to patch around. New version builds in devel/x86_64 mock and on FC6/x86_64 for me: http://wilsonet.com/packages/beryl/heliodor-0.1.2-3.fc6.src.rpm -- 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 189188] Review Request: sqlgrey - postfix grey-listing policy service
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: sqlgrey - postfix grey-listing policy service https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=189188 --- Additional Comments From [EMAIL PROTECTED] 2006-11-17 15:37 EST --- That sounds like an excellent plan to me. Let me know when you have the new versions up and ready and I can do a final checkover before approval. -- 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 215792] Review Request: wgrib - Manipulate, inventory and decode GRIB 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: wgrib - Manipulate, inventory and decode GRIB files https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=215792 [EMAIL PROTECTED] changed: What|Removed |Added OtherBugsDependingO|163778 |163779 nThis|| --- Additional Comments From [EMAIL PROTECTED] 2006-11-17 15:57 EST --- The version issue being sorted out, rpmlint is silent now, APPROVED -- 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 215569] Review Request: beryl-vidcap - Beryl OpenGL window and compositing manager video capture utility
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: beryl-vidcap - Beryl OpenGL window and compositing manager video capture utility https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=215569 --- Additional Comments From [EMAIL PROTECTED] 2006-11-17 16:31 EST --- I like the rpath-killer there, very crafty, nice to not have to resort to chrpath. I've put that in my local spec for the next build. I've also fully fixed the cflags and -devel dependencies. As for the undefined symbols... The gl* ones are indeed resolved by linking against libGL, but I'm not having any luck figuring out exactly what to do for the others. I did find that essentially the same unresolved symbols show up for all beryl and compiz plugins though... Probably going to have to poke both upstreams about this one... On the license front... I see a #define _GNU_SOURCE, but yeah, the license block is a little vague. I'll prod upstream for clarification. -- 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 212917] Review Request: php-pear-DB-DataObject - An SQL Builder, Object Interface to Database Tables
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: php-pear-DB-DataObject - An SQL Builder, Object Interface to Database Tables Alias: DB-DataObject https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=212917 [EMAIL PROTECTED] changed: What|Removed |Added OtherBugsDependingO|163778 |163779 nThis|| --- Additional Comments From [EMAIL PROTECTED] 2006-11-17 21:02 EST --- Looks good to me; rpmlint is quiet now. I'm not really sure about the symlink (I'd rather just ignore or patch the documentation) but I don't think it hurts anything. APPROVED -- 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 214024] Review Request: perl-Net-DBus - Net::DBus Perl module
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: perl-Net-DBus - Net::DBus Perl module https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=214024 [EMAIL PROTECTED] changed: What|Removed |Added OtherBugsDependingO|163778 |163779 nThis|| --- Additional Comments From [EMAIL PROTECTED] 2006-11-18 00:44 EST --- I spent a bit of time to get FC6 builds working, and builds on FC6 as well as FC5 are fine: All tests successful, 48 subtests skipped. Files=17, Tests=710, 5 wallclock secs ( 2.44 cusr + 0.44 csys = 2.88 CPU) So, rawhide builds are broken (due to problems with the packaged software, not with rawhide), but that's not a blocker. APPROVED -- 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 214236] Review Request: php-pear-SOAP - Simple Object Access Protocol (SOAP) Client/Server for PHP
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: php-pear-SOAP - Simple Object Access Protocol (SOAP) Client/Server for PHP Alias: php-pear-SOAP https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=214236 [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 215200] Review Request: php-pear-Services-Weather - This class acts as an interface to various online weather-services
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: php-pear-Services-Weather - This class acts as an interface to various online weather-services Alias: pear-Svcs-Weather https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=215200 Bug 215200 depends on bug 214236, which changed state. Bug 214236 Summary: Review Request: php-pear-SOAP - Simple Object Access Protocol (SOAP) Client/Server for PHP https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=214236 What|Old Value |New Value Resolution||NEXTRELEASE Status|ASSIGNED|CLOSED -- 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
