Bug 837668

rpmlint outputs the following:

php-doctrine-DoctrineDBAL.noarch: E: script-without-shebang /usr/bin/doctrine-dbal.php
php-doctrine-DoctrineDBAL.noarch: E: zero-length /usr/share/doc/pear/DoctrineDBAL/Doctrine/DBAL/README.markdown
php-doctrine-DoctrineDBAL.noarch: E: incorrect-fsf-address /usr/share/doc/pear/DoctrineDBAL/LICENSE
php-doctrine-DoctrineDBAL.noarch: W: no-manual-page-for-binary doctrine-dbal.php
php-doctrine-DoctrineDBAL.noarch: W: no-manual-page-for-binary doctrine-dbal

* I will fix script-without-shebang according to http://fedoraproject.org/wiki/Common_Rpmlint_issues#script-without-shebang

* Since this is a PEAR package and all files are listed in package.xml, are zero-length errors required to be fixed?

* incorrect-fsf-address: There is only whitespace and mailing address differences.  I will work with upstream to get this fixed.  Is this a blocker until it is fixed upstream?  http://fedoraproject.org/wiki/Common_Rpmlint_issues#incorrect-fsf-address is not clear if this is a blocker or not.

* I will ask upstream to fix no-manual-page-for-binary warnings

(In reply to comment #1)
> * I will fix script-without-shebang according to
> http://fedoraproject.org/wiki/Common_Rpmlint_issues#script-without-shebang

If I remove the execute mode bit from /usr/bin/doctrine-dbal.php, rpmlint gives the warning "non-executable-in-bin /usr/bin/doctrine-dbal.php 0644L".  How should this file be handled?

> * incorrect-fsf-address: There is only whitespace and mailing address
> differences.  I will work with upstream to get this fixed.  Is this a
> blocker until it is fixed upstream? 
> http://fedoraproject.org/wiki/Common_Rpmlint_issues#incorrect-fsf-address is
> not clear if this is a blocker or not.

DoctrineDBAL (actually all Doctrine projects) is changing from LGPL license to MIT license in future release.

See:
* https://groups.google.com/d/topic/doctrine-dev/1TfuNPiRvks/discussion
* http://dlm.beberlei.de/licenses/projects
* https://github.com/doctrine/dbal/commit/b3c08a7f0a1b0d8defe01448992a491481866dfc

(In reply to comment #2)
> (In reply to comment #1)
> > * I will fix script-without-shebang according to
> > http://fedoraproject.org/wiki/Common_Rpmlint_issues#script-without-shebang
> 
> If I remove the execute mode bit from /usr/bin/doctrine-dbal.php, rpmlint
> gives the warning "non-executable-in-bin /usr/bin/doctrine-dbal.php 0644L". 
> How should this file be handled?

The "source" file should include the shebang, which is

#!@php_bin@

And the package.xml should include a replacement order

<file ...
  <tasks:replace from="@php_bin@" to="php_bin" type="pear-config" />
</file>

This could be done by patching the upstream archive

A common usage is to drop the suffix for such command under linux only. You can look at "phpci" which use such solution (and also rename / ignore for .bat which is windows specific)
https://github.com/remicollet/php-compat-info/blob/master/scripts/phpci
https://github.com/remicollet/php-compat-info/blob/master/package.xml#L583
https://github.com/remicollet/php-compat-info/blob/master/package.xml#L1208

Of course, this should be proposed to upstream

> > * incorrect-fsf-address: There is only whitespace and mailing address
> > differences.  I will work with upstream to get this fixed.  Is this a
> > blocker until it is fixed upstream? 
> > http://fedoraproject.org/wiki/Common_Rpmlint_issues#incorrect-fsf-address is
> > not clear if this is a blocker or not.

This is not a blocker, You must only "report" this upstream.

> DoctrineDBAL (actually all Doctrine projects) is changing from LGPL license
> to MIT license in future release.

So, I think, it's ok to ignore this "temporary" warning.

- Updated to upstream version 2.3.0
- Added "%global pear_metadir" and usage in %install
- Added php-json require
- Updated %description
- PEAR package.xml fixes in %prep
- Changed RPM_BUILD_ROOT to %{buildroot}


Spec URL: http://siwinski.fedorapeople.org/rpmbuild/SPECS/php-doctrine-DoctrineDBAL.spec

SRPM URL: http://siwinski.fedorapeople.org/rpmbuild/SRPMS/php-doctrine-DoctrineDBAL-2.3.0-1.fc17.src.rpm

Can you please update to latest 2.3.2 before review ?
(will probably allow you to drop some hack from spec)

I don't understand why upstream want to install doctrine-dbal and doctrine-dbal.php in %{_bindir}... seems really ugly...

A single executable file, without suffix, and a correct shebang must be enough...

(In reply to Remi Collet from comment #5)
> Can you please update to latest 2.3.2 before review ?
> (will probably allow you to drop some hack from spec)

Updated to 2.3.4


(In reply to Remi Collet from comment #6)
> I don't understand why upstream want to install doctrine-dbal and
> doctrine-dbal.php in %{_bindir}... seems really ugly...
> 
> A single executable file, without suffix, and a correct shebang must be
> enough...

Changes in RPM spec to create a single executable



Spec URL: http://siwinski.fedorapeople.org/rpmbuild/SPECS/php-doctrine-DoctrineDBAL.spec

SRPM URL: http://siwinski.fedorapeople.org/rpmbuild/SRPMS/php-doctrine-DoctrineDBAL-2.3.4-1.fc18.src.rpm

Created attachment 759161 [details]
phpci.log

phpcompatinfo version 2.17.0.

Created attachment 759162 [details]
review.txt

Generated by fedora-review 0.4.1 (b2e211f) last change: 2013-04-29
Buildroot used: fedora-rawhide-x86_64
Command line :/usr/bin/fedora-review -b 837668

Minor "should"

[!]: Requires correct, justified where necessary.
	According to package.xml, PEAR >= 1.6.1
	But as EPEL-5 not targeted, this is not a blocker.

[!]: Rpmlint is run on all rpms the build produces.
	README.markdown could be removed (if upstream keep it empty)
	E: zero-length /usr/share/doc/pear/DoctrineDBAL/Doctrine/DBAL/README.markdown


No blocker

=== APPROVED ===

THANKS for the review!


New Package SCM Request
=======================
Package Name: php-doctrine-DoctrineDBAL
Short Description: Doctrine Database Abstraction Layer
Owners: siwinski
Branches: f18 f19 el6
InitialCC:

Git done (by process-git-requests).

php-doctrine-DoctrineDBAL-2.3.4-2.fc19 has been submitted as an update for Fedora 19.
https://admin.fedoraproject.org/updates/php-doctrine-DoctrineDBAL-2.3.4-2.fc19

php-doctrine-DoctrineDBAL-2.3.4-2.fc18 has been submitted as an update for Fedora 18.
https://admin.fedoraproject.org/updates/php-doctrine-DoctrineDBAL-2.3.4-2.fc18

php-doctrine-DoctrineDBAL-2.3.4-2.el6 has been submitted as an update for Fedora EPEL 6.
https://admin.fedoraproject.org/updates/php-doctrine-DoctrineDBAL-2.3.4-2.el6

php-doctrine-DoctrineDBAL-2.3.4-2.fc19 has been pushed to the Fedora 19 testing repository.

php-doctrine-DoctrineDBAL-2.3.4-2.fc18 has been pushed to the Fedora 18 stable repository.

php-doctrine-DoctrineDBAL-2.3.4-2.el6 has been pushed to the Fedora EPEL 6 stable repository.

php-doctrine-DoctrineDBAL-2.3.4-2.fc19 has been pushed to the Fedora 19 stable repository.