[Bug 209959] Review Request: tenr.de-styles-pkg - A collection of over 200 styles/themes for fluxbox

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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: tenr.de-styles-pkg - A collection of over 200 styles/themes for fluxbox


https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=209959


pertusus@xxxxxxx changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |pertusus@xxxxxxx




------- Additional Comments From pertusus@xxxxxxx  2006-10-09 07:14 EST -------
* There shouldn't be a '.' in the package name as seen here:
  http://fedoraproject.org/wiki/Packaging/NamingGuidelines

* The rpmlint issue is ignorable, although you may want to open
  a bug about missing Creative Commons licences.

* If I'm not wrong -r is impled by -a for cp.

* The Source0 should be an url. It seems to be
http://www.tenr.de/files/fluxmod-styles-pkg.tar.bz2
What is a bit bad is that there is no versionning, you 
should certainly ask upstream for a version string (in ascii
ascending order...).

Since there doesn't seems to be any release number currently,
and if upstream don't want to add any, my advice would be 
to use the stamp file to construct the version. Something along:

%define stamp 20061009

Version:        0.%{stamp}

Source0:        %{name}-%{stamp}.tar.bz2
# unversioned upstream source, downloaded with wget -N
# renamed to %%{name}-YYYYMMDD.tar.gz
#Source0:       http://www.tenr.de/files/tenr.de-styles-pkg.tar.bz2

%setup -q -n %{name}

With 
$ ls -l  tenr.de-styles-pkg.tar.bz2
-rw-r--r-- 1 dumas dumas 4778973 Oct  9 00:05 tenr.de-styles-pkg.tar.bz2

The leading 0 in version is there to try to have future version 
number higher.

* %{_styledir}/ will be unowned.

* The most annoying thing is that it doesn't work out of the box,
  the new themes don't appear anywhere in the fluxbox menu...


-- 
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@xxxxxxxxxx
http://www.redhat.com/mailman/listinfo/fedora-package-review

[Index of Archives]     [Fedora Legacy]     [Fedora Desktop]     [Fedora SELinux]     [Yosemite News]     [KDE Users]     [Fedora Tools]