Bug 459880

Is this your first package? Are you sponsored?


[rpmbuild@rocky anyremote2html]$ rpmlint -i anyremote2html-0.5.2-1.src.rpm 
anyremote2html.src:19: W: setup-not-quiet
Use the -q option to the %setup macro to avoid useless build output from
unpacking the sources.

+ rm -rf anyremote2html-0.5.2
+ /usr/bin/gzip -dc /home/rpmbuild/rpm/SOURCES/anyremote2html-0.5.2.tar.gz
+ tar -xvvf -
drwxr-xr-x fedotov/fedotov   0 2008-08-20 21:52 anyremote2html-0.5.2/
drwxr-xr-x fedotov/fedotov   0 2007-09-12 08:32 anyremote2html-0.5.2/data/
drwxr-xr-x fedotov/fedotov   0 2007-09-12 07:45 anyremote2html-0.5.2/data/16/

use -q with setup.


anyremote2html.src:32: E: files-attr-not-set
A file or a directory entry in a %files section does not have attributes set
which may result in security issues in the resulting binary package depending
on the system where the package is built.  Add default attributes using
%defattr before it in the %files section, or use per line %attr's.

anyremote2html.src:33: E: files-attr-not-set
A file or a directory entry in a %files section does not have attributes set
which may result in security issues in the resulting binary package depending
on the system where the package is built.  Add default attributes using
%defattr before it in the %files section, or use per line %attr's.

anyremote2html.src: E: description-line-too-long WEB interface for anyRemote - http://anyremote.sourceforge.net/ - remote control
Your description lines must not exceed 79 characters. If a line is exceeding
this number, cut it to fit in two lines.

anyremote2html.src: E: description-line-too-long software through bluetooth, IR or TCP/IP connection. anyRemote2html acts as HTTP
Your description lines must not exceed 79 characters. If a line is exceeding
this number, cut it to fit in two lines.

anyremote2html.src: E: no-changelogname-tag
There is no %changelog tag in your spec file. To insert it, just insert a
'%changelog' in your spec file and rebuild it.

For every build %changelog needs to be updated.


anyremote2html.src: W: strange-permission anyremote2html-0.5.2.tar.gz 0755
A file that you listed to include in your package has strange permissions.
Usually, a file should have 0644 permissions.

[rpmbuild@rocky anyremote2html]$ rpmls anyremote2html-0.5.2-1.src.rpm 
-rwxr-xr-x  anyremote2html-0.5.2.tar.gz
-rw-r--r--  anyremote2html.spec

755 is bad

1 packages and 0 specfiles checked; 5 errors, 2 warnings.

[rpmbuild@rocky noarch]$  rpmlint -i anyremote2html-0.5.2-1.noarch.rpm 
anyremote2html.noarch: W: no-documentation
The package contains no documentation (README, doc, etc). You have to include
documentation files.

If there is any documentation e.g README doc you need to include it.

(In reply to comment #1)
> Is this your first package? Are you sponsored?
This is my 4th package. Yes, i'm sponsored.

> anyremote2html.src:19: W: setup-not-quiet
Fixed.

> anyremote2html.src:32: E: files-attr-not-set
Fixed.

> anyremote2html.src: E: description-line-too-long WEB interface for anyRemote -
Fixed

> anyremote2html.src: E: no-changelogname-tag
Umm... Fixed.

> anyremote2html.src: W: strange-permission anyremote2html-0.5.2.tar.gz 0755
Fixed

> anyremote2html.noarch: W: no-documentation
Fixed

Soory for inconvinience.

Spec URL: http://downloads.sourceforge.net/anyremote/anyremote2html.spec
SRPM URL:
http://downloads.sourceforge.net/anyremote/anyremote2html-0.5.2-2.src.rpm

Is anyone interested in this package ?

(In reply to comment #2)
> http://downloads.sourceforge.net/anyremote/anyremote2html-0.5.2-2.src.rpm

Seems 404...

(In reply to comment #4) 
> Seems 404...

Oops... Mamory-san i'm very sorry.  The right link is http://downloads.sourceforge.net/anyremote/anyremote2html-0.5.2-2.fc9.src.rpm

! Some comments
  - This file is noarch, almost no actual builds are made during 
    rpmbuild, so keeping timestamps on installed files are
    highly recommended.
    So I prefer to
    * Use "cp -a" (or "cp -pr") instead of "cp -r".
    * and use 'make install DESTDIR=$RPM_BUILD_ROOT install="install -p"'
    (This is not blockers)

* %doc
  - %{_defaultdocdir}/ is marked as %doc by default so the prefix %doc is
    redundant.

* Source tarball
  - which I could download from the URL does not coincide with
    the one in your srpm.

! %changelog
  - I recommend to add hyphen before each change description like:
-----------------------------------------------------------
* Sun Aug 24 2008 Mikhail Fedotov <anyremote at mail.ru> - 0.5.2-2  
- Corrected files permissions and specs
------------------------------------------------------------
    (see: 
     http://fedoraproject.org/wiki/Packaging/Guidelines#Changelogs )
    (This is not a blocker)

(In reply to comment #6)

>     * Use "cp -a" (or "cp -pr") instead of "cp -r".
>     * and use 'make install DESTDIR=$RPM_BUILD_ROOT install="install -p"'
OK

>   - %{_defaultdocdir}/ is marked as %doc by default so the prefix %doc is
>     redundant.
OK

>   - which I could download from the URL does not coincide with
>     the one in your srpm.
Updated to coincide with each other.
 
> - Corrected files permissions and specs
OK

Updated files:
Spec URL: http://downloads.sourceforge.net/anyremote/anyremote2html.spec
SRPM URL:
http://downloads.sourceforge.net/anyremote/anyremote2html0.5.2-3.fc9.src.rpm

Well,

(In reply to comment #7)
> (In reply to comment #6)
> >   - which I could download from the URL does not coincide with
> >     the one in your srpm.
> Updated to coincide with each other.

Please don't do this way.
As the upstream you should not modify the tarball once formally released without
changing the version, unless people who want to download and compile your source by themselves
will get confused.

Please restore 0.5.2 tarball you released first and if you want to modify the
tarball itself please change the version of the tarball.

Note:
Some upstream maintainers change the tarballs in the srpms during review requests
with_out_ releasing the tarballs formally and when the reviews are passed they
release the final tarballs as formal ones.

(In reply to comment #8)

> if you want to modify the
> tarball itself please change the version of the tarball.
OK. Updated tarball and specs are here:

SRPM
http://downloads.sourceforge.net/anyremote/anyremote2html-0.5.2.1-1.src.rpm

SPEC
http://downloads.sourceforge.net/anyremote/anyremote2html.spec

TARBALL
http://downloads.sourceforge.net/anyremote/anyremote2html-0.5.2.1.tar.gz

Would you check what spec file you are using?

(In reply to comment #10)
> Would you check what spec file you are using?
Umm... I've downloaded it from specified location and checked.
Sorry, what wrong with it ? IMHO it is OK.

(In reply to comment #9)
> (In reply to comment #8)
> SRPM
> http://downloads.sourceforge.net/anyremote/anyremote2html-0.5.2.1-1.src.rpm

The spec file in this srpm
- lacks %changelog completely
- %_dist tag disappeared.
- Also this comment is ignored:
(In reply to comment #6)
>     * and use 'make install DESTDIR=$RPM_BUILD_ROOT install="install -p"'
>     (This is not blockers)

(In reply to comment #12)

> The spec file in this srpm
> - lacks %changelog completely
Oh... You are right. This srpm is not for Fedora ...

This is the right one:
http://downloads.sourceforge.net/anyremote/anyremote2html-0.5.2.1-1.fc9.src.rpm

For 0.5.2.1-1:

One issue:
* autotool automated call
-----------------------------------------------
    57  Executing(%install): /bin/sh -e /var/tmp/rpm-tmp.maoOL6
    58  + umask 022
    59  + cd /builddir/build/BUILD
    60  + cd anyremote2html-0.5.2.1
    61  + LANG=C
    62  + export LANG
    63  + unset DISPLAY
    64  + rm -rf /builddir/build/BUILDROOT/anyremote2html-0.5.2.1-1.fc10.noarch
    65  + make install DESTDIR=/builddir/build/BUILDROOT/anyremote2html-0.5.2.1-1.fc10.noarch 'install=install -p'
    66  cd . && /bin/sh /builddir/build/BUILD/anyremote2html-0.5.2.1/missing --run aclocal-1.10 
    67  /builddir/build/BUILD/anyremote2html-0.5.2.1/missing: line 54: aclocal-1.10: command not found
    68  WARNING: `aclocal-1.10' is missing on your system.  You should only need it if
    69           you modified `acinclude.m4' or `configure.in'.  You might want
    70           to install the `Automake' and `Perl' packages.  Grab them from
    71           any GNU archive site.
    72   cd . && /bin/sh /builddir/build/BUILD/anyremote2html-0.5.2.1/missing --run automake-1.10 --gnu 
-----------------------------------------------
  - It is not desired that autotools are automatically called after
    configure. Usually timestamps on some files are wrong and this
    can be fixed by "touch"ing some files.

(In reply to comment #14)
> One issue:
> * autotool automated call
Fixed.

New SRPM
http://downloads.sourceforge.net/anyremote/anyremote2html-0.5.2.2-1.fc9.src.rpm

SPEC file
http://downloads.sourceforge.net/anyremote/anyremote2html.spec

Okay.

------------------------------------------------------------
   This package (anyremote2html) is APPROVED by mtasaka
------------------------------------------------------------

(In reply to comment #16)
> Okay.
Thanks a lot. 

Umm... there is an issue here. I can not find fedora-cvs flag on this page.
Looks strange... After re-reading of http://fedoraproject.org/wiki/PackageMaintainers/CVSAdminProcedure
situation does not becane clearer.

Created attachment 316549 [details]
Place to set cvs flag

(In reply to comment #17)
> (In reply to comment #16)
> Umm... there is an issue here. I can not find fedora-cvs flag on this page.

I can see the place to set fedora-cvs flag. Would you check
if you can see the place shown in the attached file?

New Package CVS Request
=======================
Package Name: anyremote2html
Short Description: WEB interface for anyRemote
Owners: anyremote
Branches: F-8 F-9
InitialCC: anyremote

cvs done