New ebuild for Qgifer, a video-based animated GIF creator. Project looks very nice and useful, it's actively developed and hosted on sourceforge. I'd like to support this ebuild through proxy maintainer. Reproducible: Always Steps to Reproduce: 1. emerge -v media-video/qgifer 2. qgifer 3. Depends upon media-libs/giflib and media-libs/opencv
Created attachment 353260 [details] ebuild
Created attachment 353262 [details] Manifest
Created attachment 353264 [details] metadata - possible maintainer change
Created attachment 353266 [details, diff] patch
Listed you as the primary maintainer. Sorted KEYWORDS. Added dependency on dev-qt/qtgui:4 (as it is listed in `objdump -p /usr/bin/qgifer`), slotted all dependencies; added ffmpeg USE on opencv otherwise you can't open any videos which would render the program completely useless, the user however can decide which codecs are supported by changing the USE flags of ffmpeg / libav. Added virtual/ffmpeg:0 as a dependency as well, because it makes calls to it: src/converterdialog.cpp: proc->start("ffmpeg", args); src/mainwindow.h: bool checkFFMPEG(){return !QProcess::execute("ffmpeg -version");} When we look a bit further we also see the following so we depend on media-gfx/imagemagick:0 as well: src/optimizerdialog.cpp: proc->start(set->value("convert_exec","convert").toString(), args); Moved patch to PATCHES array to remove src_prepare. Added debug USE flag and src_configure phase to support debugging. Other minor changes. No idea why aidecoe is in the metadata, but if he planned to be a maintainer he'll have to add himself again; but well, I assume you copied this from somewhere. + 14 Jul 2013; Tom Wijsman <TomWij@gentoo.org> + +files/qgifer-0.2.1-desktop.patch, +metadata.xml, +qgifer-0.2.1.ebuild: + New package for media-video/qgifer, a video-based animated GIF creator. Fixed + bug #476800, reported by Andrzej Pauli (ChaosEngine) which contributed this + and will maintain it. Thank you for your contribution! If you have any questions, feel free to ask.
Excellent observations! Dindn't catch that qtgui and ffmpeg :-) However I would give options to user to choose dependencies more lightly: IUSE="debug imagemagick" ... imagemagick? ( media-gfx/imagemagick:0 ) In case of ffmpeg, there is strict dependency (as you noticed) but imagemagick is optional according to this code: qgifer-code/src/optimizerdialog.cpp > void OptimizerDialog::checkIM() > { > bool c = convertAvailable(); > if(!c) > { > msgLabel->setStyleSheet("color: red; font-size: 11pt"); > msgLabel->setText(tr("Error: this operation requires the ImageMagick installed on your system. Please install ImageMagick or choose the directory with \"convert\" executable file.")); I've attached changes into this bug PS.As of aidecoe - yes, I've copied his entries, because I thought He would be committing it to repo.
Created attachment 353276 [details] imagemagick? ( media-gfx/imagemagick:0 ) imagemagick as an option
Created attachment 353278 [details] imagemagick change in metadata.xml
(In reply to ChaosEngine from comment #6) > PS.As of aidecoe - yes, I've copied his entries, because I thought He would > be committing it to repo. Oh, where did this ebuild come from? I didn't see it when looking in overlays; unless it has been recently added somewhere, I think I didn't sync recently. Changes committed: + 14 Jul 2013; Tom Wijsman <TomWij@gentoo.org> metadata.xml, + qgifer-0.2.1.ebuild: + Made imagemagick dependency optional with USE flag imagemagick, proxied commit + for Andrzej Pauli (ChaosEngine); as remarked in bug #476800.
> Oh, where did this ebuild come from? I didn't see it when looking in overlays; unless it has been recently added somewhere, I think I didn't sync recently. It's not important at this stage. Hi is, so to speak, my local irc gentoo developer contact :-)
