[Bug 894606] Review Request: coin-or-Couenne - An exact solver for nonconvex MINLPs

[bugzilla]

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

Susi Lehtola  changed:

   What|Removed |Added

 Blocks|505154 (FE-SCITECH) |




Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=505154
[Bug 505154] Tracker: Review Requests for Science and Technology related
packages
-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org

[Bug 894606] Review Request: coin-or-Couenne - An exact solver for nonconvex MINLPs

[bugzilla]

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

Jon Ciesla limburg...@gmail.com changed:

   What|Removed |Added

  Flags|fedora-cvs? |fedora-cvs+



-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 894606] Review Request: coin-or-Couenne - An exact solver for nonconvex MINLPs

[bugzilla]

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



--- Comment #24 from Jon Ciesla limburg...@gmail.com ---
Git done (by process-git-requests).

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 894606] Review Request: coin-or-Couenne - An exact solver for nonconvex MINLPs

[bugzilla]

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

Paulo Andrade paulo.cesar.pereira.de.andr...@gmail.com changed:

   What|Removed |Added

  Flags|fedora-cvs+ |fedora-cvs?



--- Comment #23 from Paulo Andrade paulo.cesar.pereira.de.andr...@gmail.com 
---
Package Change Request
==
Package Name: coin-or-Couenne
New Branches: f21
Owners: pcpa
InitialCC: pcpa

Update coin-or stack.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 894606] Review Request: coin-or-Couenne - An exact solver for nonconvex MINLPs

[bugzilla]

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

Fedora Update System upda...@fedoraproject.org changed:

   What|Removed |Added

 Status|ON_QA   |CLOSED
   Fixed In Version||coin-or-Cbc-2.9.2-5.fc22
 Resolution|--- |ERRATA
Last Closed||2015-03-09 04:25:31



--- Comment #22 from Fedora Update System upda...@fedoraproject.org ---
coin-or-Cbc-2.9.2-5.fc22, coin-or-Ipopt-3.12.1-4.fc22,
coin-or-Bonmin-1.8.1-3.fc22, coin-or-SYMPHONY-5.6.8-1.fc22,
coin-or-FlopC++-1.1.7-4.fc22, coin-or-Couenne-0.5.2-3.fc22 has been pushed to
the Fedora 22 stable repository.  If problems still persist, please make note
of it in this bug report.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 894606] Review Request: coin-or-Couenne - An exact solver for nonconvex MINLPs

[bugzilla]

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

Fedora Update System upda...@fedoraproject.org changed:

   What|Removed |Added

 Status|MODIFIED|ON_QA



--- Comment #21 from Fedora Update System upda...@fedoraproject.org ---
Package coin-or-Cbc-2.9.2-5.fc22, coin-or-Ipopt-3.12.1-4.fc22,
coin-or-Bonmin-1.8.1-3.fc22, coin-or-SYMPHONY-5.6.8-1.fc22,
coin-or-FlopC++-1.1.7-4.fc22, coin-or-Couenne-0.5.2-3.fc22:
* should fix your issue,
* was pushed to the Fedora 22 testing repository,
* should be available at your local mirror within two days.
Update it with:
# su -c 'yum update --enablerepo=updates-testing coin-or-Cbc-2.9.2-5.fc22
coin-or-Ipopt-3.12.1-4.fc22 coin-or-Bonmin-1.8.1-3.fc22
coin-or-SYMPHONY-5.6.8-1.fc22 coin-or-FlopC++-1.1.7-4.fc22
coin-or-Couenne-0.5.2-3.fc22'
as soon as you are able to.
Please go to the following url:
https://admin.fedoraproject.org/updates/FEDORA-2015-2915/coin-or-Cbc-2.9.2-5.fc22,coin-or-Ipopt-3.12.1-4.fc22,coin-or-Bonmin-1.8.1-3.fc22,coin-or-SYMPHONY-5.6.8-1.fc22,coin-or-FlopC++-1.1.7-4.fc22,coin-or-Couenne-0.5.2-3.fc22
then log in and leave karma (feedback).

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 894606] Review Request: coin-or-Couenne - An exact solver for nonconvex MINLPs

[bugzilla]

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



--- Comment #15 from Antonio Trande anto.tra...@gmail.com ---
- Also check patch lines:

+ /usr/bin/cat /builddir/build/SOURCES/coin-or-Couenne-docdir.patch
patching file Makefile.in
Hunk #1 succeeded at 371 (offset 7 lines).
Hunk #2 succeeded at 380 (offset 7 lines).

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 894606] Review Request: coin-or-Couenne - An exact solver for nonconvex MINLPs

[bugzilla]

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



--- Comment #14 from Antonio Trande anto.tra...@gmail.com ---
- Use %{buildroot} or $RPM_BUILD_ROOT.

- Please, use %license macro.

Package Review
==

Legend:
[x] = Pass, [!] = Fail, [-] = Not applicable, [?] = Not evaluated
[ ] = Manual review needed


Issues:
===
- Package uses either %{buildroot} or $RPM_BUILD_ROOT
  Note: Using both %{buildroot} and $RPM_BUILD_ROOT
  See: http://fedoraproject.org/wiki/Packaging/Guidelines#macros


= MUST items =

C/C++:
[x]: Package does not contain kernel modules.
[x]: Package contains no static executables.
[x]: Header files in -devel subpackage, if present.
[x]: ldconfig called in %post and %postun if required.
[x]: Package does not contain any libtool archives (.la)
[x]: Rpath absent or only used for internal libs.
[x]: Development (unversioned) .so files in -devel subpackage, if present.

Generic:
[x]: Package is licensed with an open-source compatible license and meets
 other legal requirements as defined in the legal section of Packaging
 Guidelines.
[x]: License field in the package spec file matches the actual license.
 Note: Checking patched sources after %prep for licenses. Licenses found:
 Unknown or generated. 311 files have unknown license. Detailed output
 of licensecheck in /home/sagitter/894606-coin-or-Couenne/licensecheck.txt
[x]: License file installed when any subpackage combination is installed.
[x]: %build honors applicable compiler flags or justifies otherwise.
[x]: Package contains no bundled libraries without FPC exception.
[x]: Changelog in prescribed format.
[x]: Sources contain only permissible code or content.
[-]: Package contains desktop file if it is a GUI application.
[x]: Development files must be in a -devel package
[x]: Package uses nothing in %doc for runtime.
[x]: Package consistently uses macros (instead of hard-coded directory names).
[x]: Package is named according to the Package Naming Guidelines.
[x]: Package does not generate any conflict.
[x]: Package obeys FHS, except libexecdir and /usr/target.
[-]: If the package is a rename of another package, proper Obsoletes and
 Provides are present.
[x]: Requires correct, justified where necessary.
[x]: Spec file is legible and written in American English.
[-]: Package contains systemd file(s) if in need.
[x]: Useful -debuginfo package or justification otherwise.
[x]: Package is not known to require an ExcludeArch tag.
 Note: Test run failed
[x]: Large documentation must go in a -doc subpackage. Large could be size
 (~1MB) or number of files.
 Note: Test run failed
[x]: Packages must not store files under /srv, /opt or /usr/local
 Note: Test run failed
[!]: Package complies to the Packaging Guidelines
[x]: Package successfully compiles and builds into binary rpms on at least one
 supported primary architecture.
[x]: Package installs properly.
[x]: Rpmlint is run on all rpms the build produces.
 Note: There are rpmlint messages (see attachment).
[x]: If (and only if) the source package includes the text of the license(s)
 in its own file, then that file, containing the text of the license(s)
 for the package is included in %doc.
[x]: Package requires other packages for directories it uses.
[x]: Package must own all directories that it creates.
[x]: Package does not own files or directories owned by other packages.
[x]: All build dependencies are listed in BuildRequires, except for any that
 are listed in the exceptions section of Packaging Guidelines.
[x]: Package does not run rm -rf %{buildroot} (or $RPM_BUILD_ROOT) at the
 beginning of %install.
[x]: Macros in Summary, %description expandable at SRPM build time.
[x]: Package does not contain duplicates in %files.
[x]: Permissions on files are set properly.
[x]: Package use %makeinstall only when make install' ' DESTDIR=... doesn't
 work.
[x]: Package is named using only allowed ASCII characters.
[x]: Package do not use a name that already exist
[x]: Package is not relocatable.
[x]: Sources used to build the package match the upstream source, as provided
 in the spec URL.
[x]: Spec file name must match the spec package %{name}, in the format
 %{name}.spec.
[x]: File names are valid UTF-8.

= SHOULD items =

Generic:
[-]: If the source package does not include license text(s) as a separate file
 from upstream, the packager SHOULD query upstream to include it.
[x]: Final provides and requires are sane (see attachments).
[x]: Fully versioned dependency in subpackages if applicable.
 Note: No Requires: %{name}%{?_isa} = %{version}-%{release} in coin-or-
 Couenne-doc
[ ]: Package functions as described.
[x]: Latest version is packaged.
[x]: Package does not include license text files separate from upstream.
[x]: Patches link to upstream bugs/comments/lists or are otherwise justified.
[x]: Scriptlets must be sane, if used.
[-]: Description and summary sections in the 

[Bug 894606] Review Request: coin-or-Couenne - An exact solver for nonconvex MINLPs

[bugzilla]

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



--- Comment #16 from Paulo Andrade paulo.cesar.pereira.de.andr...@gmail.com 
---
Thanks! Update:

- Rediff patches (#894606#c15)
- Use license macro (#894606#c14)
- Do not mix rpm macros and rpm shell variables (#894606#c14)

Spec URL: http://pcpa.fedorapeople.org/coin-or/coin-or-Couenne.spec
SRPM URL:
http://pcpa.fedorapeople.org/coin-or/coin-or-Couenne-0.5.2-3.fc23.src.rpm

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 894606] Review Request: coin-or-Couenne - An exact solver for nonconvex MINLPs

[bugzilla]

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

Paulo Andrade paulo.cesar.pereira.de.andr...@gmail.com changed:

   What|Removed |Added

  Flags||fedora-cvs?



--- Comment #18 from Paulo Andrade paulo.cesar.pereira.de.andr...@gmail.com 
---
New Package SCM Request
===
Package Name: coin-or-Couenne
Short Description: An exact solver for nonconvex MINLPs
Upstream URL: http://www.coin-or.org/projects/Couenne.xml
Owners: pcpa
Branches: f22
InitialCC:

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 894606] Review Request: coin-or-Couenne - An exact solver for nonconvex MINLPs

[bugzilla]

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

Antonio Trande anto.tra...@gmail.com changed:

   What|Removed |Added

  Flags|fedora-review?  |fedora-review+



--- Comment #17 from Antonio Trande anto.tra...@gmail.com ---
Package approved.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 894606] Review Request: coin-or-Couenne - An exact solver for nonconvex MINLPs

[bugzilla]

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

Fedora Update System upda...@fedoraproject.org changed:

   What|Removed |Added

 Status|ASSIGNED|MODIFIED



-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 894606] Review Request: coin-or-Couenne - An exact solver for nonconvex MINLPs

[bugzilla]

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



--- Comment #20 from Fedora Update System upda...@fedoraproject.org ---
coin-or-Cbc-2.9.2-5.fc22,coin-or-Ipopt-3.12.1-4.fc22,coin-or-Bonmin-1.8.1-3.fc22,coin-or-SYMPHONY-5.6.8-1.fc22,coin-or-FlopC++-1.1.7-4.fc22,coin-or-Couenne-0.5.2-3.fc22
has been submitted as an update for Fedora 22.
https://admin.fedoraproject.org/updates/coin-or-Cbc-2.9.2-5.fc22,coin-or-Ipopt-3.12.1-4.fc22,coin-or-Bonmin-1.8.1-3.fc22,coin-or-SYMPHONY-5.6.8-1.fc22,coin-or-FlopC++-1.1.7-4.fc22,coin-or-Couenne-0.5.2-3.fc22

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 894606] Review Request: coin-or-Couenne - An exact solver for nonconvex MINLPs

[bugzilla]

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



--- Comment #19 from Jon Ciesla limburg...@gmail.com ---
Git done (by process-git-requests).

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 894606] Review Request: coin-or-Couenne - An exact solver for nonconvex MINLPs

[bugzilla]

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

Jon Ciesla limburg...@gmail.com changed:

   What|Removed |Added

  Flags|fedora-cvs? |fedora-cvs+



-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 894606] Review Request: coin-or-Couenne - An exact solver for nonconvex MINLPs

[bugzilla]

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



--- Comment #12 from Antonio Trande anto.tra...@gmail.com ---
(In reply to Antonio Trande from comment #11)
 (In reply to Paulo Andrade from comment #10)
  Update.
  
  The package now also use the ampl solver, that is
  packaged in f22 and f23, in the mp package.
  
  Spec URL: http://pcpa.fedorapeople.org/coin-or/coin-or-Couenne.spec
  SRPM URL:
  http://pcpa.fedorapeople.org/coin-or/coin-or-Couenne-0.5.2-1.fc23.src.rpm
 
 I make a new review.

Test fail:

error while loading shared libraries: libBonCouenne.so.1: cannot open shared
object file: No such file or directory

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 894606] Review Request: coin-or-Couenne - An exact solver for nonconvex MINLPs

[bugzilla]

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



--- Comment #13 from Paulo Andrade paulo.cesar.pereira.de.andr...@gmail.com 
---
Sorry. It was a side effect of changing it after
making tests and ensuring it would build, to
remove rpath. Now it sets LD_LIBRARY_PATH
during %check.

Spec URL: http://pcpa.fedorapeople.org/coin-or/coin-or-Couenne.spec
SRPM URL:
http://pcpa.fedorapeople.org/coin-or/coin-or-Couenne-0.5.2-2.fc23.src.rpm

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 894606] Review Request: coin-or-Couenne - An exact solver for nonconvex MINLPs

[bugzilla]

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



--- Comment #11 from Antonio Trande anto.tra...@gmail.com ---
(In reply to Paulo Andrade from comment #10)
 Update.
 
 The package now also use the ampl solver, that is
 packaged in f22 and f23, in the mp package.
 
 Spec URL: http://pcpa.fedorapeople.org/coin-or/coin-or-Couenne.spec
 SRPM URL:
 http://pcpa.fedorapeople.org/coin-or/coin-or-Couenne-0.5.2-1.fc23.src.rpm

I make a new review.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 894606] Review Request: coin-or-Couenne - An exact solver for nonconvex MINLPs

[bugzilla]

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



--- Comment #10 from Paulo Andrade paulo.cesar.pereira.de.andr...@gmail.com 
---
Update.

The package now also use the ampl solver, that is
packaged in f22 and f23, in the mp package.

Spec URL: http://pcpa.fedorapeople.org/coin-or/coin-or-Couenne.spec
SRPM URL:
http://pcpa.fedorapeople.org/coin-or/coin-or-Couenne-0.5.2-1.fc23.src.rpm

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 894606] Review Request: coin-or-Couenne - An exact solver for nonconvex MINLPs

[bugzilla]

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



--- Comment #6 from Antonio Trande anto.tra...@gmail.com ---
(In reply to Paulo Andrade from comment #4)
 I asked upstream about doxydoc no longer available to build on
 latest release. The tarball just bundles a pdf file.
 
 Make test is run, but does nothing if coin-or-Bonmin is not
 built with ASL support. This is a third party solver, that
 links to http://www.netlib.org/ampl/solvers/ that has no
 license information, and I am still unsure if the sources
 there are enough (there are several solvers that need a
 commercial license, and usually only provide freely a header
 file and/or some wrapper functions). I asked upstream aboout
 some clarifications about it.

Any news?

 
 Updates:
 
 - Update to latest upstream release
 - Remove module name from description
 - Create doc subpackage
 
 Spec URL: http://pcpa.fedorapeople.org/coin-or/coin-or-Couenne.spec
 SRPM URL:
 http://pcpa.fedorapeople.org/coin-or/coin-or-Couenne-0.4.7-1.fc22.src.rpm



Package Review
==

Legend:
[x] = Pass, [!] = Fail, [-] = Not applicable, [?] = Not evaluated
[ ] = Manual review needed



= MUST items =

C/C++:
[x]: Package does not contain kernel modules.
[x]: Package contains no static executables.
[x]: Header files in -devel subpackage, if present.
[x]: ldconfig called in %post and %postun if required.
[x]: Package does not contain any libtool archives (.la)
[x]: Rpath absent or only used for internal libs.
[x]: Development (unversioned) .so files in -devel subpackage, if present.

Generic:
[x]: Package is licensed with an open-source compatible license and meets
 other legal requirements as defined in the legal section of Packaging
 Guidelines.
[x]: License field in the package spec file matches the actual license.
 Note: Checking patched sources after %prep for licenses. Licenses found:
 Unknown or generated. 325 files have unknown license. Detailed output
 of licensecheck in /home/sagitter/894606-coin-or-Couenne/licensecheck.txt
[x]: License file installed when any subpackage combination is installed.
[x]: %build honors applicable compiler flags or justifies otherwise.
[x]: Package contains no bundled libraries without FPC exception.
[x]: Changelog in prescribed format.
[x]: Sources contain only permissible code or content.
[-]: Package contains desktop file if it is a GUI application.
[x]: Development files must be in a -devel package
[x]: Package uses nothing in %doc for runtime.
[x]: Package consistently uses macros (instead of hard-coded directory names).
[x]: Package is named according to the Package Naming Guidelines.
[x]: Package does not generate any conflict.
[x]: Package obeys FHS, except libexecdir and /usr/target.
[-]: If the package is a rename of another package, proper Obsoletes and
 Provides are present.
[x]: Requires correct, justified where necessary.
[x]: Spec file is legible and written in American English.
[-]: Package contains systemd file(s) if in need.
[x]: Useful -debuginfo package or justification otherwise.
[x]: Package is not known to require an ExcludeArch tag.
[x]: Large documentation must go in a -doc subpackage. Large could be size
 (~1MB) or number of files.
 Note: Documentation size is 20480 bytes in 4 files.
[x]: Package complies to the Packaging Guidelines
[x]: Package successfully compiles and builds into binary rpms on at least one
 supported primary architecture.
[x]: Package installs properly.
[x]: Rpmlint is run on all rpms the build produces.
 Note: There are rpmlint messages (see attachment).
[x]: If (and only if) the source package includes the text of the license(s)
 in its own file, then that file, containing the text of the license(s)
 for the package is included in %doc.
[x]: Package requires other packages for directories it uses.
[x]: Package must own all directories that it creates.
[x]: Package does not own files or directories owned by other packages.
[x]: All build dependencies are listed in BuildRequires, except for any that
 are listed in the exceptions section of Packaging Guidelines.
[x]: Package uses either %{buildroot} or $RPM_BUILD_ROOT
[x]: Package does not run rm -rf %{buildroot} (or $RPM_BUILD_ROOT) at the
 beginning of %install.
[x]: Macros in Summary, %description expandable at SRPM build time.
[x]: Package does not contain duplicates in %files.
[x]: Permissions on files are set properly.
[x]: Package use %makeinstall only when make install' ' DESTDIR=... doesn't
 work.
[x]: Package is named using only allowed ASCII characters.
[x]: Package do not use a name that already exist
[x]: Package is not relocatable.
[x]: Sources used to build the package match the upstream source, as provided
 in the spec URL.
[x]: Spec file name must match the spec package %{name}, in the format
 %{name}.spec.
[x]: File names are valid UTF-8.
[x]: Packages must not store files under /srv, /opt or /usr/local

= SHOULD items =

[Bug 894606] Review Request: coin-or-Couenne - An exact solver for nonconvex MINLPs

[bugzilla]

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



--- Comment #7 from Antonio Trande anto.tra...@gmail.com ---
Created attachment 942309
  -- https://bugzilla.redhat.com/attachment.cgi?id=942309action=edit
Full review.txt file

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 894606] Review Request: coin-or-Couenne - An exact solver for nonconvex MINLPs

[bugzilla]

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



--- Comment #8 from Paulo Andrade paulo.cesar.pereira.de.andr...@gmail.com ---
(In reply to Antonio Trande from comment #6)
 (In reply to Paulo Andrade from comment #4)
  I asked upstream about doxydoc no longer available to build on
  latest release. The tarball just bundles a pdf file.
  
  Make test is run, but does nothing if coin-or-Bonmin is not
  built with ASL support. This is a third party solver, that
  links to http://www.netlib.org/ampl/solvers/ that has no
  license information, and I am still unsure if the sources
  there are enough (there are several solvers that need a
  commercial license, and usually only provide freely a header
  file and/or some wrapper functions). I asked upstream aboout
  some clarifications about it.
 
 Any news?

You should have received some emails saturday, where I pinged
Ted Ralphs and CC'ed you. But I think at first the only real
blocker, if any, is the documentation issue. Extra solver should
require significant work, somewhat like MUMPS for coin-or-Ipopt,
and someone that actually uses it, being interested enough to
contribute to the packaging (too much of these 3rd party
solvers have either non-free license, or just do not
provide source code).

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 894606] Review Request: coin-or-Couenne - An exact solver for nonconvex MINLPs

[bugzilla]

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



--- Comment #9 from Antonio Trande anto.tra...@gmail.com ---
(In reply to Paulo Andrade from comment #8)
 (In reply to Antonio Trande from comment #6)
  (In reply to Paulo Andrade from comment #4)
   I asked upstream about doxydoc no longer available to build on
   latest release. The tarball just bundles a pdf file.
   
   Make test is run, but does nothing if coin-or-Bonmin is not
   built with ASL support. This is a third party solver, that
   links to http://www.netlib.org/ampl/solvers/ that has no
   license information, and I am still unsure if the sources
   there are enough (there are several solvers that need a
   commercial license, and usually only provide freely a header
   file and/or some wrapper functions). I asked upstream aboout
   some clarifications about it.
  
  Any news?
 
 You should have received some emails saturday, where I pinged
 Ted Ralphs and CC'ed you. But I think at first the only real
 blocker, if any, is the documentation issue. 

Just one mail about FlopC++.


 Extra solver should require significant work, somewhat like MUMPS for coin-or-
 Ipopt, and someone that actually uses it, being interested enough to
 contribute to the packaging (too much of these 3rd party
 solvers have either non-free license, or just do not
 provide source code).

Is there an extra-solver that we can package at the moment?

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 894606] Review Request: coin-or-Couenne - An exact solver for nonconvex MINLPs

[bugzilla]

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



--- Comment #5 from Antonio Trande anto.tra...@gmail.com ---
(In reply to Paulo Andrade from comment #4)
 I asked upstream about doxydoc no longer available to build on
 latest release. The tarball just bundles a pdf file.
 
 Make test is run, but does nothing if coin-or-Bonmin is not
 built with ASL support. This is a third party solver, that
 links to http://www.netlib.org/ampl/solvers/ that has no
 license information, and I am still unsure if the sources
 there are enough (there are several solvers that need a
 commercial license, and usually only provide freely a header
 file and/or some wrapper functions). I asked upstream aboout
 some clarifications about it.


I wait some days for these replies before to conclude the review.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 894606] Review Request: coin-or-Couenne -An exact solver for nonconvex MINLPs

[bugzilla]

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

Paulo Andrade paulo.cesar.pereira.de.andr...@gmail.com changed:

   What|Removed |Added

Summary|Review Request: |Review Request:
   |coin-or-Couenne -Couenne,   |coin-or-Couenne -An exact
   |an exact solver for |solver for nonconvex MINLPs
   |nonconvex MINLPs|



-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 894606] Review Request: coin-or-Couenne - An exact solver for nonconvex MINLPs

[bugzilla]

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

Paulo Andrade paulo.cesar.pereira.de.andr...@gmail.com changed:

   What|Removed |Added

Summary|Review Request: |Review Request:
   |coin-or-Couenne -An exact   |coin-or-Couenne - An exact
   |solver for nonconvex MINLPs |solver for nonconvex MINLPs



-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 894606] Review Request: coin-or-Couenne - An exact solver for nonconvex MINLPs

[bugzilla]

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



--- Comment #4 from Paulo Andrade paulo.cesar.pereira.de.andr...@gmail.com ---
I asked upstream about doxydoc no longer available to build on
latest release. The tarball just bundles a pdf file.

Make test is run, but does nothing if coin-or-Bonmin is not
built with ASL support. This is a third party solver, that
links to http://www.netlib.org/ampl/solvers/ that has no
license information, and I am still unsure if the sources
there are enough (there are several solvers that need a
commercial license, and usually only provide freely a header
file and/or some wrapper functions). I asked upstream aboout
some clarifications about it.

Updates:

- Update to latest upstream release
- Remove module name from description
- Create doc subpackage

Spec URL: http://pcpa.fedorapeople.org/coin-or/coin-or-Couenne.spec
SRPM URL:
http://pcpa.fedorapeople.org/coin-or/coin-or-Couenne-0.4.7-1.fc22.src.rpm

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review