[Bug 3887] Review request: display-dhammapada - Displays a verse from the Dhammapada

[RPM Fusion Bugzilla]

https://bugzilla.rpmfusion.org/show_bug.cgi?id=3887

Nicolas Chauvet  changed:

   What|Removed |Added

 Blocks|3   |


Referenced Bugs:

https://bugzilla.rpmfusion.org/show_bug.cgi?id=3
[Bug 3] Tracker: Packages under review.
-- 
You are receiving this mail because:
You are on the CC list for the bug.___
rpmfusion-developers mailing list -- rpmfusion-developers@lists.rpmfusion.org
To unsubscribe send an email to rpmfusion-developers-le...@lists.rpmfusion.org

[Bug 3887] Review request: display-dhammapada - Displays a verse from the Dhammapada

[RPM Fusion Bugzilla]

https://bugzilla.rpmfusion.org/show_bug.cgi?id=3887

--- Comment #5 from bodhi  ---
Sorry I did not see the original message.

I am more then happy to make the modifications suggested to the package or if
preferred migrate to fortune.

Please discuss and let me know what direction to go in.

-- 
You are receiving this mail because:
You are on the CC list for the bug.

[Bug 3887] Review request: display-dhammapada - Displays a verse from the Dhammapada

[RPM Fusion Bugzilla]

https://bugzilla.rpmfusion.org/show_bug.cgi?id=3887

Dominik 'Rathann' Mierzejewski  changed:

   What|Removed |Added

 CC||domi...@greysector.net

--- Comment #4 from Dominik 'Rathann' Mierzejewski  ---
fortune-mod has always been in Fedora:
https://apps.fedoraproject.org/packages/fortune-mod

-- 
You are receiving this mail because:
You are on the CC list for the bug.

[Bug 3887] Review request: display-dhammapada - Displays a verse from the Dhammapada

[RPM Fusion Bugzilla]

http://bugzilla.rpmfusion.org/show_bug.cgi?id=3887

Nicolas Chauvet  changed:

   What|Removed |Added

 Status|ASSIGNED|RESOLVED
 Resolution|--- |EXPIRED

--- Comment #2 from Nicolas Chauvet  ---
No news, please re-open when updated.

-- 
You are receiving this mail because:
You are on the CC list for the bug.

[Bug 3887] Review request: display-dhammapada - Displays a verse from the Dhammapada

[RPM Fusion Bugzilla]

http://bugzilla.rpmfusion.org/show_bug.cgi?id=3887

Nicolas Chauvet  changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 Blocks|2   |3
   Assignee|rpmfusion-package-review@rp |kwiz...@gmail.com
   |mfusion.org |

--- Comment #1 from Nicolas Chauvet  ---
While public domain is probably correct for the original content, your software
to display it is licensed as GPLv3(+ ?), this should appear in the license
field of the RPM with (idealy) a correct license file added to the upstream
project.

* Please fix the indent of the description (avoid space in the beginning of new
lines and at the top. And use the full allowed 80 characters.

* Drop unneeded %define _hardened_build 1 (this is the default nowadays)

* Use %make_build and %make_install macros

* Drop uneeded defattr

* Verify that all your directories are owned by the package.

* Use wildcard for manpages compression extension.

* There are spurious spaces in the beginning of some sections, please remove
and keep 2 spaces at the end of earch section.

So to me this should be provided in the free section, do you agree ?

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are the assignee for the bug.