[Bug 2216] Review request: Not Tetris 2 - Classic Tetris mixed with physics
https://bugzilla.rpmfusion.org/show_bug.cgi?id=2216
--- Comment #10 from Jeremy Newton 2012-04-06 20:51:24
CEST ---
(In reply to comment #9)
> Sorry for the delay.
>
> (In reply to comment #3)
> > A bunch of notes, I can do a full review later:
> > [...]
>
> Great help, thanks. I followed almost all of these, and your SRPM was of great
No problem, always up to help :)
> > -You missing %post, %postun and %posttrans sections, which are required for
> > desktop files with icons: (you can also see my SPRM below)
> > https://fedoraproject.org/wiki/Packaging/Guidelines#Icon_tag_in_Desktop_Files
>
> Well, in https://fedoraproject.org/wiki/Packaging:ScriptletSnippets#Icon_Cache
> says that is to do when 'an application installs icons into one of the
> subdirectories in %{_datadir}/icons/', but that's not the case anymore. Seeing
> your SRPM example, I wrote the icon into %_datadir/pixmaps/, so the previous
> condition is not fulfilled anymore. For that reason, I did not add those
> scriptlets. Correct me if I'm wrong, please.
I believe pixmaps is also a part of the icon cache, thus it still needs to be
updated, though I am not completely sure on this. I'll let you know when I
figure that out. I guess I just always assumed it should be there without
checking.
> > By the way, kudos for the idea of getting the logo from the love file, it
> > never
> > occurred to me to do that.
>
> Thanks. If you are thinking about packaging mari0, just mention that you can
> do
> that there too. If not, I can package it.
I have mari0 packaged, I just need to upload it. I got love accepted into
Fedora, so I just want to run a quick test before I upload it.
> > -Man pages are not vital for getting your package accepted, but it is
> > suggested
> > to look into making one.
>
> I have not a clue about how to do this. I think this should be upstream work,
> though.
It's very simple, you can make one in a few minutes. Take a look at my love
package for an example:
http://kojipkgs.fedoraproject.org/packages/love/0.8.0/2.fc18/src/love-0.8.0-2.fc18.src.rpm
Just open the love.1 with a text editor and the rest is easy.
You can either make one or ask upstream to make one, which I highly doubt they
will. If you do make one, you can send it upstream and they may include it in
the linux version though.
I would like to strongly note that you are not required to make a man page,
though it's just highly suggested.
> > -To my knowledge %defattr(-, root, root) shouldn't be required, but to be
> > honest I've never used it before.
>
> I never really understood that. I used to put it because I saw it in a lot of
> places. Anyway I removed that.
I believe that is for compatibility with RedHat EL 5, and unless you plan to
package for EPEL 5, it's pointless. As well I'm only currently maintaining love
for Fedora 15, 16, 17, devel/rawhide
On another note, the version of love I submitted for fedora, which is still in
updates testing, is version 0.8.0. Though they have not advertised it on their
website yet, love 0.8.0 has been tagged in their SCM (bitbucket). I'm not sure
what this is all about.
Anyway, Not tetris at the moment does not work on 0.8.0, and I contacted
upstream about this. They said until love 0.8.0 is "official released" they
don't plan to update Not Tetris.
Naturally this issue puts this review on hold unless you would like to take a
crack at making a patch yourself or would like to contact upstream for one.
--
Configure bugmail: https://bugzilla.rpmfusion.org/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
You are the assignee for the bug.
[Bug 2216] Review request: Not Tetris 2 - Classic Tetris mixed with physics
https://bugzilla.rpmfusion.org/show_bug.cgi?id=2216
--- Comment #9 from Yajo 2012-04-06 19:24:42 CEST ---
Sorry for the delay.
(In reply to comment #3)
> A bunch of notes, I can do a full review later:
> [...]
Great help, thanks. I followed almost all of these, and your SRPM was of great
help. Just a few points:
> -You missing %post, %postun and %posttrans sections, which are required for
> desktop files with icons: (you can also see my SPRM below)
> https://fedoraproject.org/wiki/Packaging/Guidelines#Icon_tag_in_Desktop_Files
Well, in https://fedoraproject.org/wiki/Packaging:ScriptletSnippets#Icon_Cache
says that is to do when 'an application installs icons into one of the
subdirectories in %{_datadir}/icons/', but that's not the case anymore. Seeing
your SRPM example, I wrote the icon into %_datadir/pixmaps/, so the previous
condition is not fulfilled anymore. For that reason, I did not add those
scriptlets. Correct me if I'm wrong, please.
> By the way, kudos for the idea of getting the logo from the love file, it
> never
> occurred to me to do that.
Thanks. If you are thinking about packaging mari0, just mention that you can do
that there too. If not, I can package it.
> -Man pages are not vital for getting your package accepted, but it is
> suggested
> to look into making one.
I have not a clue about how to do this. I think this should be upstream work,
though.
> -To my knowledge %defattr(-, root, root) shouldn't be required, but to be
> honest I've never used it before.
I never really understood that. I used to put it because I saw it in a lot of
places. Anyway I removed that.
Here is my new SRPM: http://www.mediafire.com/?82odbzgh9jr8vc6
--
Configure bugmail: https://bugzilla.rpmfusion.org/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
You are the assignee for the bug.
[Bug 2216] Review request: Not Tetris 2 - Classic Tetris mixed with physics
https://bugzilla.rpmfusion.org/show_bug.cgi?id=2216 Yajo changed: What|Removed |Added Blocks||30 -- Configure bugmail: https://bugzilla.rpmfusion.org/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. You are the assignee for the bug.
[Bug 30] Tracker : Sponsorship Request
https://bugzilla.rpmfusion.org/show_bug.cgi?id=30 Yajo changed: What|Removed |Added Depends on||2216 -- Configure bugmail: https://bugzilla.rpmfusion.org/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the assignee for the bug.
[Bug 2127] Review Request: libdvbcsa - DVB Common Scrambling Algorithm with encryption and decryption capabilities
https://bugzilla.rpmfusion.org/show_bug.cgi?id=2127 --- Comment #3 from Nicolas Chauvet 2012-04-06 17:25:00 CEST --- I've found a component that use libdvbcsa, also some patches to use libdvbcsa within vlc have been published recently. https://github.com/gfto/tsdecrypt -- Configure bugmail: https://bugzilla.rpmfusion.org/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. You are the assignee for the bug.
Re: [Bug 2113] smplayer 0.8.0 and smtube 1.1 released
Hi, On Fri, 2012-04-06 at 02:21 +, RPM Fusion Bugzilla wrote: > New releases comes very fast now. smplayer-0.8.0 and smtube-1.1 released. > http://downloads.sourceforge.net/sourceforge/smplayer/smplayer-0.8.0.tar.bz2 > http://downloads.sourceforge.net/sourceforge/smplayer/smtube-1.1.tar.bz2 > > Both using bundled qtsingleapplication, so attaching patch for using system > one. Nucleo , proposes add smtube to smplayer package. smtube provides : possible to search and download videos from youtube. So I'm asking to the list if any problem on include that, at least at point of view of "legal stuff", but I see that video downloaders is not a problem ... Thanks, -- Sérgio M. B.
[Bug 2261] Review request geotrans-3.1
https://bugzilla.rpmfusion.org/show_bug.cgi?id=2261 Dan Horák changed: What|Removed |Added CC||d...@danny.cz --- Comment #1 from Dan Horák 2012-04-06 09:00:23 CEST --- Hm, unless I'm missing something then the license is free and thus acceptable for Fedora (IANAL). ... NGA hereby grants to each user of the software a license to use and distribute the software, and develop derivative works. => rights for use, distributing and modifying are granted which are the main attributes making the license acceptable -- Configure bugmail: https://bugzilla.rpmfusion.org/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. You are the assignee for the bug.
