[Bug 907007] New: Review Request: unittest-cpp - Lightweight unit testing framework for C++
Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=907007 Bug ID: 907007 Summary: Review Request: unittest-cpp - Lightweight unit testing framework for C++ Product: Fedora Version: rawhide Component: Package Review Severity: medium Priority: medium Reporter: f...@fcami.net Spec URL: http://fcami.fedorapeople.org/srpms/unittest-cpp.spec SRPM URL: http://fcami.fedorapeople.org/srpms/unittest-cpp-1.4-1.fc18.src.rpm Description: UnitTest++ is a lightweight unit testing framework for C++. Simplicity, portability, speed, and small footprint are all very important aspects of UnitTest++. Fedora Account System Username: fcami Notes: * scratch build: http://koji.fedoraproject.org/koji/taskinfo?taskID=4923797 * Upstream builds static-only, I have switched to a shared library but will not choose a soname in upstream's place. * I have asked upstream to provide a shared library: https://github.com/charlesnicholson/unittest-cpp/issues/24 -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=kENWzlg7MYa=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 907007] Review Request: unittest-cpp - Lightweight unit testing framework for C++
Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=907007 François Cami f...@fcami.net changed: What|Removed |Added Whiteboard||NotReady --- Comment #1 from François Cami f...@fcami.net --- Please do not review yet. -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=mZX1XeX6ywa=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 904658] Review Request: radium-compressor: an audio compressor for JACK
Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=904658 Mamoru TASAKA mtas...@fedoraproject.org changed: What|Removed |Added Status|NEW |ASSIGNED Assignee|nob...@fedoraproject.org|mtas...@fedoraproject.org Flags||fedora-review? --- Comment #1 from Mamoru TASAKA mtas...@fedoraproject.org --- For 0.5.1-1 * License - Some files are under GPLv3+, and some of them are actually used to build the binary: - audio/faudiostream/architecture/faust/audio/alsa-dsp.h audio/faudiostream/architecture/faust/audio/coreaudio-dsp.h audio/faudiostream/architecture/faust/audio/netjack-dsp.h audio/faudiostream/architecture/faust/audio/portaudio-dsp.h audio/faudiostream/architecture/faust/gui/console.h audio/faudiostream/architecture/faust/gui/faustqt.h audio/faudiostream/architecture/faust/gui/meta.h faudiostream/architecture/faust/audio/alsa-dsp.h faudiostream/architecture/faust/audio/coreaudio-dsp.h faudiostream/architecture/faust/audio/netjack-dsp.h faudiostream/architecture/faust/audio/portaudio-dsp.h faudiostream/architecture/faust/gui/console.h faudiostream/architecture/faust/gui/faustqt.h faudiostream/architecture/faust/gui/meta.h - So the license tag should be GPLv3+ (just GPLv3+ is okay). * Compiler flags - Fedora specific compilation flags are not used correctly. build.log says: - 33 Executing(%build): /bin/sh -e /var/tmp/rpm-tmp.ESDXyl 34 + umask 022 35 + cd /builddir/build/BUILD 36 + cd radium_compressor-0.5.1 37 + make 'OPTS=-O2 -g -pipe -Wall -Wp,-D_FORTIFY_SOURCE=2 -fexceptions -fstack-protector --param=ssp-buffer-size=4 -m32 -march=i686 -mtune=atom -fasynchronous-unwind-tables' -j5 38 cd Qt ./create_source_from_ui.sh `../find_moc_and_uic_paths.sh uic` `../find_moc_and_uic_paths.sh moc` compressor_widget 39 g++ -DDEBUG -O3 -Wall -msse -mfpmath=sse -DUSE_QT_REQTYPE -DUSE_QT4 -g -I. -ffast-math -IQt Qt/Qt_SliderPainter.cpp `pkg-config --cflags Qt3Support` -c 40 g++ -DDEBUG -O3 -Wall -msse -mfpmath=sse -DUSE_QT_REQTYPE -DUSE_QT4 -g -I. -ffast-math -IQt main.cpp Qt_SliderPainter.o -Iaudio/faudiostream/architecture/ `pkg-config --libs --cflags Qt3Support` -ljack -o radium_compressor 41 + exit 0 - Also, -O3 is used, this should be changed to -O2. And, is it safe for this package to use -ffast-math? -ffast-math does some optimization which breaks IEEE rules for example, which may result in confusing behavior. * rpmlint issue - incorrect-fsf-address (not a blocker) - Some source files contain incorrest FSF address, which causes rpmlint complaint on debuginfo rpm. Not a blocker, however it is recommend to tell upstream - spurious-executable-perm - rpmlint complains about some header files in debuginfo rpm for having non-proper permission. You can suppress this rpmlint complaint by changing the permission of header files to 0644 at %prep. * Desktop file https://fedoraproject.org/wiki/Packaging:Guidelines?rd=Packaging/Guidelines#Desktop_files - It seems that radium_compressor is a GUI program, so installing proper desktop file is required (unless there is some reason, e.g. this program usually needs proper argument, for example) -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=CSA0xMvA70a=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 907020] New: Review Request: stbi - JPEG/PNG reader
Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=907020 Bug ID: 907020 Summary: Review Request: stbi - JPEG/PNG reader Product: Fedora Version: rawhide Component: Package Review Severity: medium Priority: medium Reporter: mhron...@redhat.com Spec URL: https://raw.github.com/hroncok/SPECS/master/stbi.spec SRPM URL: https://github.com/downloads/hroncok/SPECS/stbi-1.33-1.fc18.src.rpm Description: Public Domain JPEG/PNG reader. Primarily of interest to game developers and other people who can avoid problematic images and only need the trivial interface. Fedora Account System Username: churchyard -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=9aoxLljNI1a=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 903829] Review Request: perl-Time-Interval - Convert seconds to human readable form
Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=903829 --- Comment #12 from Normunds l...@rule.lv --- Big thanks for your support with getting response from upstream. Updated to version 1.232, added obfuscated e-mail, added LICENSE to doc. Spec URL: http://unibackup.rule.lv/FedoraRPM/perl-Time-Interval.spec SRPM URL: http://unibackup.rule.lv/FedoraRPM/perl-Time-Interval-1.232-1.fc16.src.rpm Thanks. -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=GbQMaORngsa=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 887769] Review Request: lv2-newtonator - an LV2 soft synth using velocity and acceleration
Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=887769 Mamoru TASAKA mtas...@fedoraproject.org changed: What|Removed |Added Status|NEW |ASSIGNED Assignee|nob...@fedoraproject.org|mtas...@fedoraproject.org Flags||fedora-review? --- Comment #1 from Mamoru TASAKA mtas...@fedoraproject.org --- Looks good, will check in detail later. By the way, build fails on F-18 although you seem to have created your srpm on F-18, is it okay? (build on F-19 is fine): http://koji.fedoraproject.org/koji/taskinfo?taskID=4923947 -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=B4m1lMHG9ia=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 904328] Review Request: perl-Config-ApacheFormat - Use Apache format config files
Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=904328 --- Comment #9 from Normunds l...@rule.lv --- [...] wasn't autodetected in this case. perl(Class::MethodMaker), on the other hand, is. Why did you add it now? Still getting used to rpmbuild behavior. I was assuming that if rpmbuild detects Requirement and requirement is specified in spec file, it will show duplicate Requirement in rpm --requires output. * perl(Class::MethodMaker) removed from Requirements. * Added obfuscated e-mail. Thanks. -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=AYLrl6bG7La=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 887750] Review Request: lv2-fomp-plugins - a collection of LV2 plugins ported from LADSPA
Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=887750 Mamoru TASAKA mtas...@fedoraproject.org changed: What|Removed |Added Status|NEW |ASSIGNED Assignee|nob...@fedoraproject.org|mtas...@fedoraproject.org Flags||fedora-review? --- Comment #1 from Mamoru TASAKA mtas...@fedoraproject.org --- Taking. -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=KY4jv9CYy9a=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 907027] New: Review Request: rapidxml - Fast XML parser
Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=907027 Bug ID: 907027 Summary: Review Request: rapidxml - Fast XML parser Product: Fedora Version: rawhide Component: Package Review Severity: medium Priority: medium Reporter: mhron...@redhat.com Spec URL: https://raw.github.com/hroncok/SPECS/master/rapidxml.spec SRPM URL: https://github.com/downloads/hroncok/SPECS/rapidxml-1.13-1.fc18.src.rpm Description: RapidXml is an attempt to create the fastest XML parser possible, while retaining usability, portability and reasonable W3C compatibility. It is an in-situ parser written in modern C++, with parsing speed approaching that of strlen function executed on the same data. Fedora Account System Username: churchyard -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=ryCRJrig8Ba=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 907018] New: Review Request: stbi - JPEG/PNG reader
Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=907018 Bug ID: 907018 Summary: Review Request: stbi - JPEG/PNG reader Product: Fedora Version: rawhide Component: Package Review Severity: medium Priority: medium Reporter: mhron...@redhat.com Spec URL: https://raw.github.com/hroncok/SPECS/master/stbi.spec SRPM URL: https://github.com/downloads/hroncok/SPECS/stbi-1.33-1.fc18.src.rpm Description: Public Domain JPEG/PNG reader. Primarily of interest to game developers and other people who can avoid problematic images and only need the trivial interface. Fedora Account System Username: churchyard --- Comment #1 from Miro Hrončok mhron...@redhat.com --- *** Bug 907020 has been marked as a duplicate of this bug. *** -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=hlXMC7G14ta=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 907020] Review Request: stbi - JPEG/PNG reader
Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=907020 Miro Hrončok mhron...@redhat.com changed: What|Removed |Added Status|NEW |CLOSED Resolution|--- |DUPLICATE Last Closed||2013-02-02 14:10:31 --- Comment #1 from Miro Hrončok mhron...@redhat.com --- Sorry, a mistake. *** This bug has been marked as a duplicate of bug 907018 *** -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=bPXB4E4n8ya=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 907032] New: Review Request: amftools - AMF file library
Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=907032 Bug ID: 907032 Summary: Review Request: amftools - AMF file library Product: Fedora Version: rawhide Component: Package Review Severity: medium Priority: medium Reporter: mhron...@redhat.com Spec URL: https://raw.github.com/hroncok/SPECS/master/amftools.spec SRPM URL: https://github.com/downloads/hroncok/SPECS/amftools-0.0-1.20121220svn32.fc18.src.rpm Description: C++ tools for implementing AMF file format for the interchange of geometry for 3D printing (additive manufacturing). Fedora Account System Username: churchyard -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=B8S9qnlQ6va=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 907020] Review Request: stbi - JPEG/PNG reader
Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=907020 Miro Hrončok mhron...@redhat.com changed: What|Removed |Added Blocks||907032 -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=KE5xE9YGnFa=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 907032] Review Request: amftools - AMF file library
Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=907032 Miro Hrončok mhron...@redhat.com changed: What|Removed |Added Depends On||907020, 907027 -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=HF80MpmwtHa=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 907027] Review Request: rapidxml - Fast XML parser
Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=907027 Miro Hrončok mhron...@redhat.com changed: What|Removed |Added Blocks||907032 -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=AU5O69BRC2a=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 907018] Review Request: stbi - JPEG/PNG reader
Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=907018 Miro Hrončok mhron...@redhat.com changed: What|Removed |Added Blocks||907032 -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=7b67ADaagqa=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 907020] Review Request: stbi - JPEG/PNG reader
Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=907020 Miro Hrončok mhron...@redhat.com changed: What|Removed |Added Blocks|907032 | -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=RxEZUNL9aga=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 907032] Review Request: amftools - AMF file library
Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=907032 Miro Hrončok mhron...@redhat.com changed: What|Removed |Added Depends On|907020 |907018 -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=MYobMa9MiZa=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 848990] Review Request: libnetfilter_acct - A library providing interface to extended accounting infrastructure
Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=848990 --- Comment #29 from Fedora Update System upda...@fedoraproject.org --- libnetfilter_acct-1.0.0-2.el6 has been pushed to the Fedora EPEL 6 stable repository. -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=DQw4WyAOqAa=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 907036] New: Review Request: perl-Math-PlanePath - Points on a path through the 2D plane
Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=907036 Bug ID: 907036 Summary: Review Request: perl-Math-PlanePath - Points on a path through the 2D plane Product: Fedora Version: rawhide Component: Package Review Severity: medium Priority: medium Reporter: mhron...@redhat.com Spec URL: https://raw.github.com/hroncok/SPECS/master/perl-Math-PlanePath.spec SRPM URL: https://github.com/downloads/hroncok/SPECS/perl-Math-PlanePath-98-1.fc18.src.rpm Description: This is a base class for some mathematical paths which map an integer position $n to and from coordinates $x,$y in the 2D plane. Fedora Account System Username: churchyard -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=Bp6m3h77xAa=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 890839] Review Request: slic3r - G-code generator for 3D printers (RepRap, Makerbot, Ultimaker etc.)
Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=890839 Miro Hrončok mhron...@redhat.com changed: What|Removed |Added Depends On||907036 -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=9CymsrDiaNa=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 890839] Review Request: slic3r - G-code generator for 3D printers (RepRap, Makerbot, Ultimaker etc.)
Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=890839 Bug 890839 depends on bug 876928, which changed state. Bug 876928 Summary: Review Request: perl-Math-PlanePath - Points on a path through the 2-D plane https://bugzilla.redhat.com/show_bug.cgi?id=876928 What|Removed |Added Status|ASSIGNED|CLOSED Resolution|--- |DUPLICATE -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=ZGwqTVwn0Ua=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 876928] Review Request: perl-Math-PlanePath - Points on a path through the 2-D plane
Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=876928 Miro Hrončok mhron...@redhat.com changed: What|Removed |Added Status|ASSIGNED|CLOSED Resolution|--- |DUPLICATE Flags|needinfo?(m...@v3.sk) | Last Closed||2013-02-02 14:49:33 --- Comment #11 from Miro Hrončok mhron...@redhat.com --- *** This bug has been marked as a duplicate of bug 907036 *** -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=8hDaMdIBTHa=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 907036] Review Request: perl-Math-PlanePath - Points on a path through the 2D plane
Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=907036 Miro Hrončok mhron...@redhat.com changed: What|Removed |Added Blocks||890839 CC||m...@v3.sk --- Comment #1 from Miro Hrončok mhron...@redhat.com --- *** Bug 876928 has been marked as a duplicate of this bug. *** -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=88Nf367xvDa=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 890839] Review Request: slic3r - G-code generator for 3D printers (RepRap, Makerbot, Ultimaker etc.)
Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=890839 Bug 890839 depends on bug 887913, which changed state. Bug 887913 Summary: Review Request: perl-Math-Clipper - Polygon clipping in 2D https://bugzilla.redhat.com/show_bug.cgi?id=887913 What|Removed |Added Status|MODIFIED|CLOSED Resolution|--- |CURRENTRELEASE -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=YozHsmpbeXa=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 887913] Review Request: perl-Math-Clipper - Polygon clipping in 2D
Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=887913 Miro Hrončok mhron...@redhat.com changed: What|Removed |Added Status|MODIFIED|CLOSED Resolution|--- |CURRENTRELEASE Last Closed||2013-02-02 14:50:32 -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=hcUKIpSCrPa=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 894725] Review Request: uglify-js - JavaScript parser, mangler/compressor and beautifier toolkit
Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=894725 --- Comment #16 from Vít Ondruch vondr...@redhat.com --- I am afraid that the -common package does not contain the uglifyjs executable, so it is worthless for rubygem-uglifier purposes. Moreover, the uglifier gem contains just single uglify.js file, which probably contains all the uglifyjs files. Not sure how to cope with that. If linking will be enough. -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=uNnDDEQB29a=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 904658] Review Request: radium-compressor: an audio compressor for JACK
Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=904658 --- Comment #2 from Kjetil Matheussen k.s.matheus...@notam02.no --- As far as I can read from http://gcc.gnu.org/onlinedocs/gcc-4.7.0/gcc/Optimize-Options.html, -ffast-math should be safe, and benchmarking showed that -ffast-math makes the program run 13% faster. (I'm the upstream author) -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=1EFUMUWi8la=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 894725] Review Request: uglify-js - JavaScript parser, mangler/compressor and beautifier toolkit
Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=894725 --- Comment #17 from T.C. Hollingsworth tchollingswo...@gmail.com --- (In reply to comment #16) I am afraid that the -common package does not contain the uglifyjs executable, so it is worthless for rubygem-uglifier purposes. I can probably accommodate this, but I can't figure out where rubygem-uglifier actually uses it. It doesn't appear to bundle it. If you can point me to Ruby code that calls it in, that would be helpful. Moreover, the uglifier gem contains just single uglify.js file, which probably contains all the uglifyjs files. Not sure how to cope with that. If linking will be enough. Looking at the Rakefile [1], it seems that it bundles source-map and uglify-js by compiling them both into one .js file. There is no bundled libraries exception for JavaScript not in the browser, so I don't think this is permissible in Fedora. rubygem-uglifier will have to be patched to bring in source-map and uglify-js separately. I can, however, ship a minified version of uglify-js like rubygem-uglifier wants, though I'm not sure why it's necessary. The whole point of minification is to reduce JS loading time in the browser; it's completely unnecessary server-side. [1] https://github.com/lautis/uglifier/blob/master/Rakefile -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=FUnJiLS6nza=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 904658] Review Request: radium-compressor: an audio compressor for JACK
Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=904658 --- Comment #3 from Kjetil Matheussen k.s.matheus...@notam02.no --- Oh, I also see that -O3 makes the program run twice as fast as -O2 (at least on my computer): make benchmark ./benchmark. It's -fpredictive-commoning and -ftree-vectorize that makes the difference. -O2 -fpredictive-commoning -ftree-vectorize runs at the same speed as -O3. -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=MZ343x3SZka=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 906764] Review Request: manaplus - OpenSource 2D MMORPG client for Evol Online and The Mana World
Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=906764 --- Comment #4 from Erik Schilling ablu.erikschill...@googlemail.com --- Spec: ablu.fedorapeople.org/SRPMS/manaplus-1.3.2.3-1.fc18.src.rpm SRPM: http://ablu.fedorapeople.org/manaplus-1.3.2.3-1.fc18.src.rpm koji: http://koji.fedoraproject.org/koji/taskinfo?taskID=4924268 * Sun Feb 3 2013 Erik Schilling ablu.erikschill...@googlemail.com 1.3.2.3-1 - New upstream release 1.3.2.3 -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=sOTGYo5DsTa=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 906764] Review Request: manaplus - OpenSource 2D MMORPG client for Evol Online and The Mana World
Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=906764 --- Comment #5 from Erik Schilling ablu.erikschill...@googlemail.com --- Spec: http://ablu.fedorapeople.org/manaplus.spec Sorry. Silly copy paste error. -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=ZpuCE8EphAa=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 773487] Review Request: trac-batchmodify-plugin - Batch modification of tickets
Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=773487 --- Comment #7 from Kevin Fenzi ke...@scrye.com --- Shall we push this to stable now? -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=pVwJEoZsmqa=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 803317] Review Request: trac-themeengine-plugin - Simple API and GUI for packaging and configuring Trac themes
Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=803317 --- Comment #7 from Kevin Fenzi ke...@scrye.com --- Shall we push this to stable now? -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=7lCBteiO1ma=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 832504] Review Request: pesign - Utility for signing UEFI applications
Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=832504 Kevin Fenzi ke...@scrye.com changed: What|Removed |Added Status|ASSIGNED|CLOSED Resolution|--- |RAWHIDE Last Closed||2013-02-02 22:00:39 --- Comment #17 from Kevin Fenzi ke...@scrye.com --- This is in and done, closing. -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=vXhqsEIzVYa=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 894551] Review Request: compat-libffi - A portable foreign function interface library
Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=894551 Kevin Fenzi ke...@scrye.com changed: What|Removed |Added Status|ASSIGNED|CLOSED Resolution|--- |RAWHIDE Last Closed||2013-02-02 22:01:47 --- Comment #6 from Kevin Fenzi ke...@scrye.com --- This is in and done. closing. -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=wxlPcsVpwPa=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 905681] Review Request: ultimaker-marlin-firmware - Ultimaker firmware for the 3D printer
Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=905681 --- Comment #3 from Miro Hrončok mhron...@redhat.com --- There is /usr/lib/firmware/ dir, shouldn't that be used instead of /usr/share? -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=ssSFy26QYCa=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 904658] Review Request: radium-compressor: an audio compressor for JACK
Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=904658 --- Comment #4 from Mamoru TASAKA mtas...@fedoraproject.org --- (In reply to comment #3) Oh, I also see that -O3 makes the program run twice as fast as -O2 (at least on my computer): make benchmark ./benchmark. It's -fpredictive-commoning and -ftree-vectorize that makes the difference. -O2 -fpredictive-commoning -ftree-vectorize runs at the same speed as -O3. Well, -O2 vs -O3 is rather Fedora packaging (policy) issue. I can accept -O2 -fpredictive-commoning -ftree-vectorize instead. -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=A5XPmbmmd7a=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 864187] Review Request: openscad - The Programmers Solid 3D CAD Modeller
Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=864187 Fedora Update System upda...@fedoraproject.org changed: What|Removed |Added CC|ryanskingsbury@alumni.utexa | |s.net | --- Comment #45 from Fedora Update System upda...@fedoraproject.org --- openscad-2013.01.17-3.fc17 has been submitted as an update for Fedora 17. https://admin.fedoraproject.org/updates/openscad-2013.01.17-3.fc17 -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=JdgTlpOVKNa=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 864187] Review Request: openscad - The Programmers Solid 3D CAD Modeller
Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=864187 --- Comment #46 from Fedora Update System upda...@fedoraproject.org --- openscad-2013.01.17-3.fc18 has been submitted as an update for Fedora 18. https://admin.fedoraproject.org/updates/openscad-2013.01.17-3.fc18 -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=yNrY7ihECLa=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 907075] New: Review Request: qaccessibilityclient - QAccessibilityClient library is used when writing accessibility clients
Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=907075 Bug ID: 907075 Summary: Review Request: qaccessibilityclient - QAccessibilityClient library is used when writing accessibility clients Product: Fedora Version: rawhide Component: Package Review Severity: medium Priority: medium Reporter: rdie...@math.unl.edu Spec URL: http://rdieter.fedorapeople.org/rpms/simon/qaccessibilityclient.spec SRPM URL: http://rdieter.fedorapeople.org/rpms/simon/qaccessibilityclient-0.1.0-0.1.20121113git.fc18.src.rpm Description: QAccessibilityClient library is used when writing accessibility clients Fedora Account System Username: rdieter1 -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=4duZrNCgWMa=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 907075] Review Request: qaccessibilityclient - QAccessibilityClient library is used when writing accessibility clients
Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=907075 Rex Dieter rdie...@math.unl.edu changed: What|Removed |Added Depends On||656997 (kde-reviews) Alias||qaccessibilityclient -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=BrIFjEat9Ua=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 907076] New: Review Request: simon - Open source speech recognition
Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=907076 Bug ID: 907076 Summary: Review Request: simon - Open source speech recognition Product: Fedora Version: rawhide Component: Package Review Severity: medium Priority: medium Reporter: rdie...@math.unl.edu Spec URL: http://rdieter.fedorapeople.org/rpms/simon/simon.spec SRPM URL: http://rdieter.fedorapeople.org/rpms/simon/simon-0.4.0-1.fc18.src.rpm Description: Open source speech recognition Fedora Account System Username: rdieter1 -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=Qlg3eA4mLja=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 907076] Review Request: simon - Open source speech recognition
Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=907076 Rex Dieter rdie...@math.unl.edu changed: What|Removed |Added Depends On||907075 ||(qaccessibilityclient) Alias||simon --- Comment #1 from Rex Dieter rdie...@math.unl.edu --- make that, Fedora Account System Username: rdieter -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=GVeJP5gRN5a=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 907075] Review Request: qaccessibilityclient - QAccessibilityClient library is used when writing accessibility clients
Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=907075 Rex Dieter rdie...@math.unl.edu changed: What|Removed |Added Blocks||907076 (simon) -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=fJuNyA83uta=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 904658] Review Request: radium-compressor: an audio compressor for JACK
Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=904658 --- Comment #5 from Mamoru TASAKA mtas...@fedoraproject.org --- (In reply to comment #3) Oh, I also see that -O3 makes the program run twice as fast as -O2 (at least on my computer): make benchmark ./benchmark. It's -fpredictive-commoning and -ftree-vectorize that makes the difference. -O2 -fpredictive-commoning -ftree-vectorize runs at the same speed as -O3. Oh, https://fedoraproject.org/wiki/Packaging:Guidelines?rd=Packaging/Guidelines#Compiler_flags says Overriding these flags for performance optimizations (for instance, -O3 instead of -O2) is generally discouraged. If you can present benchmarks that show a significant speedup for this particular code, this could be revisited on a case-by-case basis. So as you kindly presented benckmarks, I can accept -O3 and -ffast-math. Still other Fedora specific compilation flags (such as -Wp,-D_FORTIFY_SOURCE=2 and so on) must be honored. -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=14FKp7jIQua=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 907076] Review Request: simon - Open source speech recognition
Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=907076 Kevin Kofler ke...@tigcc.ticalc.org changed: What|Removed |Added Blocks||656997 (kde-reviews) -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=bPFUhvzXm9a=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 907075] Review Request: qaccessibilityclient - QAccessibilityClient library is used when writing accessibility clients
Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=907075 Kevin Kofler ke...@tigcc.ticalc.org changed: What|Removed |Added Blocks||656997 (kde-reviews) Depends On|656997 (kde-reviews)| -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=8lUSAHrgXqa=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 907075] Review Request: qaccessibilityclient - QAccessibilityClient library is used when writing accessibility clients
Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=907075 --- Comment #1 from Kevin Kofler ke...@tigcc.ticalc.org --- As in the other review, the Fedora Account System Username should be just rdieter, not rdieter1. :-) -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=6uicNv3tria=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 887769] Review Request: lv2-newtonator - an LV2 soft synth using velocity and acceleration
Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=887769 --- Comment #2 from Mamoru TASAKA mtas...@fedoraproject.org --- Okay, (while I don't know well about lv2) the packaging for this package is okay. If it is expected that your srpm don't build on F-18, I can approve this. -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=WZecRpjgloa=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review