[Bug 894594] Review Request: coin-or-Bcp - Branch-Cut-Price Framework
https://bugzilla.redhat.com/show_bug.cgi?id=894594 Fedora Update System upda...@fedoraproject.org changed: What|Removed |Added Status|ON_QA |CLOSED Fixed In Version||coin-or-Bcp-1.3.8-1.fc20 Resolution|--- |ERRATA Last Closed||2014-05-01 03:01:57 --- Comment #11 from Fedora Update System upda...@fedoraproject.org --- coin-or-Bcp-1.3.8-1.fc20 has been pushed to the Fedora 20 stable repository. -- 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 894594] Review Request: coin-or-Bcp - Branch-Cut-Price Framework
https://bugzilla.redhat.com/show_bug.cgi?id=894594 Fedora Update System upda...@fedoraproject.org changed: What|Removed |Added Status|MODIFIED|ON_QA --- Comment #10 from Fedora Update System upda...@fedoraproject.org --- coin-or-Bcp-1.3.8-1.fc20 has been pushed to the Fedora 20 testing repository. -- 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 894594] Review Request: coin-or-Bcp - Branch-Cut-Price Framework
https://bugzilla.redhat.com/show_bug.cgi?id=894594 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 894594] Review Request: coin-or-Bcp - Branch-Cut-Price Framework
https://bugzilla.redhat.com/show_bug.cgi?id=894594 --- Comment #9 from Fedora Update System upda...@fedoraproject.org --- coin-or-Bcp-1.3.8-1.fc20 has been submitted as an update for Fedora 20. https://admin.fedoraproject.org/updates/coin-or-Bcp-1.3.8-1.fc20 -- 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 894594] Review Request: coin-or-Bcp - Branch-Cut-Price Framework
https://bugzilla.redhat.com/show_bug.cgi?id=894594 Kevin Fenzi ke...@scrye.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 894594] Review Request: coin-or-Bcp - Branch-Cut-Price Framework
https://bugzilla.redhat.com/show_bug.cgi?id=894594 --- Comment #8 from Kevin Fenzi ke...@scrye.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 894594] Review Request: coin-or-Bcp - Branch-Cut-Price Framework
https://bugzilla.redhat.com/show_bug.cgi?id=894594 Paulo Andrade paulo.cesar.pereira.de.andr...@gmail.com changed: What|Removed |Added Flags||fedora-cvs? --- Comment #7 from Paulo Andrade paulo.cesar.pereira.de.andr...@gmail.com --- New Package SCM Request === Package Name: coin-or-Bcp Short Description: Branch-Cut-Price Framework Owners: pcpa Branches: f20 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 894594] Review Request: coin-or-Bcp - Branch-Cut-Price Framework
https://bugzilla.redhat.com/show_bug.cgi?id=894594 Bug 894594 depends on bug 894593, which changed state. Bug 894593 Summary: Review Request: coin-or-Vol - Vol (Volume Algorithm) https://bugzilla.redhat.com/show_bug.cgi?id=894593 What|Removed |Added Status|ON_QA |CLOSED Resolution|--- |ERRATA -- 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 894594] Review Request: coin-or-Bcp - Branch-Cut-Price Framework
https://bugzilla.redhat.com/show_bug.cgi?id=894594 Bug 894594 depends on bug 894587, which changed state. Bug 894587 Summary: Review Request: coin-or-Clp - Coin-or linear programming https://bugzilla.redhat.com/show_bug.cgi?id=894587 What|Removed |Added Status|ON_QA |CLOSED Resolution|--- |ERRATA -- 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 894594] Review Request: coin-or-Bcp - Branch-Cut-Price Framework
https://bugzilla.redhat.com/show_bug.cgi?id=894594
Antonio Trande anto.tra...@gmail.com changed:
What|Removed |Added
Flags|fedora-review? |fedora-review+
--- Comment #6 from Antonio Trande anto.tra...@gmail.com ---
Directory /usr/include/coin owned by coin-or-CoinUtils-devel package; properly
listed.
Package approved.
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. 225 files have unknown license. Detailed output
of licensecheck in /home/sagitter/894594-coin-or-Bcp/licensecheck.txt
[x]: License file installed when any subpackage combination is installed.
[x]: Package requires other packages for directories it uses.
Note: No known owner of /usr/include/coin
[x]: Package must own all directories that it creates.
Note: Directories without known owners: /usr/include/coin
[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 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 =
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).
[ ]: 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]:
[Bug 894594] Review Request: coin-or-Bcp - Branch-Cut-Price Framework
https://bugzilla.redhat.com/show_bug.cgi?id=894594 Antonio Trande anto.tra...@gmail.com changed: What|Removed |Added Status|NEW |ASSIGNED Assignee|nob...@fedoraproject.org|anto.tra...@gmail.com Flags||fedora-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 894594] Review Request: coin-or-Bcp - Branch-Cut-Price Framework
https://bugzilla.redhat.com/show_bug.cgi?id=894594 --- Comment #5 from Paulo Andrade paulo.cesar.pereira.de.andr...@gmail.com --- Update: - Update to latest upstream release. Spec URL: http://pcpa.fedorapeople.org/coin-or/coin-or-Bcp.spec SRPM URL: http://pcpa.fedorapeople.org/coin-or/coin-or-Bcp-1.3.8-1.fc21.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 894594] Review Request: coin-or-Bcp - Branch-Cut-Price Framework
https://bugzilla.redhat.com/show_bug.cgi?id=894594 Bug 894594 depends on bug 894586, which changed state. Bug 894586 Summary: Review Request: coin-or-Osi - COIN-OR Open Solver Interface Library https://bugzilla.redhat.com/show_bug.cgi?id=894586 What|Removed |Added Status|ON_QA |CLOSED Resolution|--- |ERRATA -- 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 894594] Review Request: coin-or-Bcp - Branch-Cut-Price Framework
https://bugzilla.redhat.com/show_bug.cgi?id=894594 Bug 894594 depends on bug 894588, which changed state. Bug 894588 Summary: Review Request: coin-or-Cgl - Cut Generation Library https://bugzilla.redhat.com/show_bug.cgi?id=894588 What|Removed |Added Status|ASSIGNED|CLOSED Resolution|--- |RAWHIDE -- 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 894594] Review Request: coin-or-Bcp - Branch-Cut-Price Framework
https://bugzilla.redhat.com/show_bug.cgi?id=894594 --- Comment #3 from Paulo Andrade paulo.cesar.pereira.de.andr...@gmail.com --- - Update to latest upstream release. Spec URL: http://pcpa.fedorapeople.org/coin-or/coin-or-Bcp.spec SRPM URL: http://pcpa.fedorapeople.org/coin-or/coin-or-Bcp-1.3.6-1.fc21.src.rpm -- You are receiving this mail because: You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 894594] Review Request: coin-or-Bcp - Branch-Cut-Price Framework
https://bugzilla.redhat.com/show_bug.cgi?id=894594 --- Comment #4 from Paulo Andrade paulo.cesar.pereira.de.andr...@gmail.com --- Update: - Use proper _smp_flags macro (#894586#c6). Spec URL: http://pcpa.fedorapeople.org/coin-or/coin-or-Bcp.spec SRPM URL: http://pcpa.fedorapeople.org/coin-or/coin-or-Bcp-1.3.6-2.fc21.src.rpm -- You are receiving this mail because: You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 894594] Review Request: coin-or-Bcp - Branch-Cut-Price Framework
Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=894594 Bug 894594 depends on bug 894585, which changed state. Bug 894585 Summary: Review Request: coin-or-CoinUtils - Coin-or Utilities https://bugzilla.redhat.com/show_bug.cgi?id=894585 What|Removed |Added Status|ON_QA |CLOSED Resolution|--- |ERRATA -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=BPkGFtNhoma=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 894594] Review Request: coin-or-Bcp - Branch-Cut-Price Framework
Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=894594 --- Comment #2 from Paulo Andrade paulo.cesar.pereira.de.andr...@gmail.com --- Update: - Update to run make check (#894610#c4). Spec URL: http://pcpa.fedorapeople.org/coin-or/coin-or-Bcp.spec SRPM URL: http://pcpa.fedorapeople.org/coin-or/coin-or-Bcp-1.3.4-4.fc19.src.rpm -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=Av5XdCCvqla=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 894594] Review Request: coin-or-Bcp - Branch-Cut-Price Framework
Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=894594 Paulo Andrade paulo.cesar.pereira.de.andr...@gmail.com changed: What|Removed |Added Depends On||894585, 894586, 894587, ||894588, 894593 --- Comment #1 from Paulo Andrade paulo.cesar.pereira.de.andr...@gmail.com --- Note that the tarball is remade due to: + Data files without a clean license. licensecheck does not trigger it because they are small test case files, but a not so small collection, and authorship information was lost. + ThirdParty directory, that points to, but has no contents, of non free code (usually source code open but needs some kind of paid license to be able to use). + Most coin-or projects bundle other coin-or projects that are dependencies. If tarballs are not repackaged, %build will remove the bundled dependencies. I made the original package back in september and was talking from time to time to upstream about the issues above. There should be at some point in the near future a new release with bundled dependencies and code that cannot be redistributed removed from tarballs. There is also a way to get clean tarballs from coin-or trac, but for the review request I did choose the most common method in Fedora for these conditions. -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=1pXpi1BsSra=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 894594] Review Request: coin-or-Bcp - Branch-Cut-Price Framework
Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=894594 Paulo Andrade paulo.cesar.pereira.de.andr...@gmail.com changed: What|Removed |Added Blocks||894609 -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=h29BSgOJY2a=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
