Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
Summary: Review Request: dolibarr - ERP & CRM for small companies, freelances or foundations
https://bugzilla.redhat.com/show_bug.cgi?id=723326
Summary: Review Request: dolibarr - ERP & CRM for small companies, freelances or foundations Product: Fedora Version: rawhide Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: unspecified Component: Package Review AssignedTo: nobody@fedoraproject.org ReportedBy: eldy@destailleur.fr QAContact: extras-qa@fedoraproject.org CC: notting@redhat.com, package-review@lists.fedoraproject.org Classification: Fedora Story Points: ---
Spec URL: http://www.dolibarr.org/files/fedora/dolibarr.spec SRPM URL: http://www.dolibarr.org/files/fedora/dolibarr-3.1.0-0.src.rpm Description: Dolibarr ERP & CRM is an easy to use open source/free software for small and medium companies, foundations or freelances. It includes different features for Enterprise Resource Planning (ERP) and Customer Relationship Management (CRM) but also for different other activities.
I am author of other already packaged OpenSources packages (awstats) but this is my first package to maintain myself. So i'm seeking a sponsor.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=723326
Laurent Destailleur eldy@destailleur.fr changed:
What |Removed |Added ---------------------------------------------------------------------------- Blocks| |177841(FE-NEEDSPONSOR)
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=723326
--- Comment #1 from Laurent Destailleur eldy@destailleur.fr 2011-07-31 15:17:13 EDT --- I have resubmitted SRPMS to match package names rules and fix the target arch. Correct one is "noarch".
This is links to SRPMS to use is: http://www.dolibarr.org/files/fedora/dolibarr-3.1.0-0.2.beta1.src.rpm
And links to binary package is: http://www.dolibarr.org/files/fedora/dolibarr-3.1.0-0.2.beta1.noarch.rpm
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=723326
--- Comment #2 from Remi Collet fedora@famillecollet.com 2011-08-06 08:22:02 EDT --- First comments
Don't define name, version, release (Name, Version and Release directives are enough)
Release must include %{?dist}
For readilibilty clean the spec of all non-fedora stuff
Requiring mysql-server is probably a bad idea (you can install the web-app on a server and the DB on another)
Most stuff in %post should go to %install (and %file) Most stuff in %postun should be obsoleted by previous comment
Don't create /usr/share/doc/dolibar. Use %doc <pathinsourcetree> File will be added to /usr/share/doc/dolibar-3.1.0 which is the standard default location.
Lang file must be tag as lang file.
You must not bundle library (artichow, tcpdf, ..) but use system ones (and submit new packages if not available).
Use desktop-file-install to install .desktop (not install)
Please confirm the requires PHP extensions / version required (phpci reports php 5.3.0 and lot extensions, probably some "false/positive" due to embedded libraries)
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=723326
--- Comment #3 from Laurent Destailleur eldy@destailleur.fr 2011-09-04 17:03:59 EDT --- I uploaded a new version with the following changes: * spec include %{?dist} * move creation of conf files into the install * removed creation of doc folder /usr/share/doc/dolibarr. * language files use the name.lang * use desktop-file-install instead of .desktop * change php requirement to 5.3.0 * removed all embedded ressources already available as a package (so no more font files are provided, and removed all changes done into libraries) * also the package is built from the standard tgz distribution.
This is links to SRPMS to use is: http://www.dolibarr.org/files/fedora/dolibarr-3.2.0-0.1.a.fc15.src.rpm
And links to binary package is: http://www.dolibarr.org/files/fedora/dolibarr-3.2.0-0.1.a.fc15.noarch.rpm
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=723326
--- Comment #4 from Laurent Destailleur eldy@destailleur.fr 2011-10-27 11:54:58 EDT --- Last build with tgz source file, binary and src rpm is now available at: http://www.dolibarr.org/files/lastbuild/package_rpm_redhat-fedora/
Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=723326
Jason Tibbitts tibbs@math.uh.edu changed:
What |Removed |Added ---------------------------------------------------------------------------- Flags| |needinfo?(eldy@destailleur. | |fr)
--- Comment #5 from Jason Tibbitts tibbs@math.uh.edu --- I am triaging old review tickets. I can't promise a review if you reply, but by closing out the stale tickets we can devote extra attention to the ones which aren't stale.
Please post a spec file so that people can look at without chasing down your srpm and unpacking it.
Note a few rpmlint complaints:
dolibarr.noarch: W: invalid-license GPL-3.0+ Please use a valid License: tag from https://fedoraproject.org/wiki/Licensing:Main
dolibarr.noarch: E: incorrect-fsf-address /usr/share/dolibarr/htdocs/webservices/demo_wsclient_category.php-NORUN dolibarr.noarch: E: incorrect-fsf-address /usr/share/dolibarr/htdocs/cron/info.php dolibarr.noarch: E: incorrect-fsf-address /usr/share/dolibarr/htdocs/webservices/server_category.php Might want to fix these upstream.
dolibarr.noarch: E: non-executable-script /usr/share/dolibarr/htdocs/includes/odtphp/odt2pdf.sh 0644L /bin/bash Why is there a shell script under htdocs?
dolibarr.noarch: W: dangerous-command-in-%post rm dolibarr.noarch: W: dangerous-command-in-%postun rm You really shouldn't do this kind of thing. I unpacked the srpm to look at the scriptlets, and they are pretty much totally insane. scriptlets cannot produce output. You should include a sane configuration file in the package and not mess with it on package upgrades. The selinux stuff on the surface is OK, though if this package gets in you should communicate with the selinux team to get the necessary contexts into the default policy. Also, I don't think httpd_sys_script_rw_t is a valid context. Your scriptlets to restart things are completely improper. And I'm out of breath now.
Throughout the spec you use macro forms for commands like "%{__chmod}". These should not be used; just say "chmod" instead.
That's about all I have time for at the moment; there are plenty of other issues I can see, though.
https://bugzilla.redhat.com/show_bug.cgi?id=723326
Laurent Destailleur eldy@destailleur.fr changed:
What |Removed |Added ---------------------------------------------------------------------------- Flags|needinfo?(eldy@destailleur. | |fr) |
--- Comment #6 from Laurent Destailleur eldy@destailleur.fr --- This is link that includes all spec files and sources for all versions: http://www.dolibarr.org/files/stable/package_rpm_redhat-fedora/
https://bugzilla.redhat.com/show_bug.cgi?id=723326
William Moreno williamjmorenor@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- CC| |williamjmorenor@gmail.com
--- Comment #7 from William Moreno williamjmorenor@gmail.com --- Any update here? Dollibar is a ERP than could be nice to get in Fedora, any way to help here?
https://bugzilla.redhat.com/show_bug.cgi?id=723326
Miroslav Suchý msuchy@redhat.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |CLOSED CC| |msuchy@redhat.com Blocks|177841 (FE-NEEDSPONSOR) | Resolution|--- |DEFERRED Last Closed| |2015-07-21 09:14:42
--- Comment #8 from Miroslav Suchý msuchy@redhat.com --- Closing due long inactivity. Feel free to reopen if you want to continue.
Referenced Bugs:
https://bugzilla.redhat.com/show_bug.cgi?id=177841 [Bug 177841] Tracker: Review requests from new Fedora packagers who need a sponsor
package-review@lists.fedoraproject.org