package-review April 2015

package-review@lists.fedoraproject.org

1 participants
1852 discussions

[Bug 921134] Review Request: boost148 - The free peer-reviewed portable C++ source libraries
by Red Hat Bugzilla 12 Apr '15

12 Apr '15

https://bugzilla.redhat.com/show_bug.cgi?id=921134 Jon Ciesla <limburgher(a)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

1 0

[Bug 921134] Review Request: boost148 - The free peer-reviewed portable C++ source libraries
by Red Hat Bugzilla 12 Apr '15

12 Apr '15

https://bugzilla.redhat.com/show_bug.cgi?id=921134 --- Comment #25 from Denis Arnaud <denis.arnaud_fedora(a)m4x.org> --- Successful builds on EPEL 6 for all the architectures: http://koji.fedoraproject.org/koji/taskinfo?taskID=9463313 -- 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

1 0

[Bug 921134] Review Request: boost148 - The free peer-reviewed portable C++ source libraries
by Red Hat Bugzilla 11 Apr '15

11 Apr '15

https://bugzilla.redhat.com/show_bug.cgi?id=921134 --- Comment #24 from Denis Arnaud <denis.arnaud_fedora(a)m4x.org> --- Successful build on EPEL 6: http://koji.fedoraproject.org/koji/taskinfo?taskID=9463281 -- 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

1 0

[Bug 921134] Review Request: boost148 - The free peer-reviewed portable C++ source libraries
by Red Hat Bugzilla 11 Apr '15

11 Apr '15

https://bugzilla.redhat.com/show_bug.cgi?id=921134 --- Comment #23 from Denis Arnaud <denis.arnaud_fedora(a)m4x.org> --- (In reply to Denis Arnaud from comment #21) > (In reply to Rex Dieter from comment #20) > > The el6 failure is an undefined macro: > > + '%{_mpich2_load}' > > /var/tmp/rpm-tmp.pLSGl7: line 97: fg: no job control > > The only thing I do not understand, then, is why it used to work before. For > instance, > http://denisarnaud.fedorapeople.org/boost/boost148/boost148-1.48.0-4.spec > contains the exact same macro (%{_mpich2_load})... > As a matter of fact, MPICH2 has been upgraded into MPICH v3. So, the macro is now simply named _mpich_load (the '2' suffix has been dropped). -- 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

1 0

[Bug 921134] Review Request: boost148 - The free peer-reviewed portable C++ source libraries
by Red Hat Bugzilla 11 Apr '15

11 Apr '15

https://bugzilla.redhat.com/show_bug.cgi?id=921134 Denis Arnaud <denis.arnaud_fedora(a)m4x.org> 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

1 0

[Bug 921134] Review Request: boost148 - The free peer-reviewed portable C++ source libraries
by Red Hat Bugzilla 11 Apr '15

11 Apr '15

https://bugzilla.redhat.com/show_bug.cgi?id=921134 Denis Arnaud <denis.arnaud_fedora(a)m4x.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Flags| |fedora-cvs+ --- Comment #22 from Denis Arnaud <denis.arnaud_fedora(a)m4x.org> --- New Package SCM Request ======================= Package Name: boost148 Short Description: The free peer-reviewed portable C++ source libraries Upstream URL: http://www.boost.org/users/history/version_1_48_0.html Owners: denisarnaud Branches: f20 f21 f22 el5 el6 epel7 InitialCC: rdieter -- 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

1 0

[Bug 921134] Review Request: boost148 - The free peer-reviewed portable C++ source libraries
by Red Hat Bugzilla 11 Apr '15

11 Apr '15

https://bugzilla.redhat.com/show_bug.cgi?id=921134 --- Comment #21 from Denis Arnaud <denis.arnaud_fedora(a)m4x.org> --- (In reply to Rex Dieter from comment #20) > The el6 failure is an undefined macro: > + '%{_mpich2_load}' > /var/tmp/rpm-tmp.pLSGl7: line 97: fg: no job control Thanks! The only thing I do not understand, then, is why it used to work before. For instance, http://denisarnaud.fedorapeople.org/boost/boost148/boost148-1.48.0-4.spec contains the exact same macro (%{_mpich2_load})... -- 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

1 0

[Bug 921134] Review Request: boost148 - The free peer-reviewed portable C++ source libraries
by Red Hat Bugzilla 11 Apr '15

11 Apr '15

https://bugzilla.redhat.com/show_bug.cgi?id=921134 Rex Dieter <rdieter(a)math.unl.edu> changed: What |Removed |Added ---------------------------------------------------------------------------- Flags|fedora-review? |fedora-review+ --- Comment #20 from Rex Dieter <rdieter(a)math.unl.edu> --- The el6 failure is an undefined macro: + '%{_mpich2_load}' /var/tmp/rpm-tmp.pLSGl7: line 97: fg: no job control I won't treat it as a review blocker here, since the epel-7 build succeeds, but obviously this needs to be fixed if you want to support epel-6 Thanks. 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

1 0

[Bug 921134] Review Request: boost148 - The free peer-reviewed portable C++ source libraries
by Red Hat Bugzilla 11 Apr '15

11 Apr '15

https://bugzilla.redhat.com/show_bug.cgi?id=921134 --- Comment #19 from Denis Arnaud <denis.arnaud_fedora(a)m4x.org> --- EPEL 7: http://koji.fedoraproject.org/koji/taskinfo?taskID=9463024 The builds were successful for both x86-64 and ppc64 (there is an error at the end in Koji, but it seems to be a Koji issue, as all the RPMs have successfully been built) --------------------- So, ready for approval? -- 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

1 0

[Bug 921134] Review Request: boost148 - The free peer-reviewed portable C++ source libraries
by Red Hat Bugzilla 11 Apr '15

11 Apr '15

https://bugzilla.redhat.com/show_bug.cgi?id=921134 --- Comment #18 from Denis Arnaud <denis.arnaud_fedora(a)m4x.org> --- Spec URL: http://denisarnaud.fedorapeople.org/boost/boost148/boost148.spec SRPM URL: http://denisarnaud.fedorapeople.org/boost/boost148/boost148-1.48.0-5.fc20.s… EPEL 7: http://koji.fedoraproject.org/koji/taskinfo?taskID=9462960 EPEL 6: http://koji.fedoraproject.org/koji/taskinfo?taskID=9462986 (the build is fine, but the Koji build process fails at the end because of a lack of resource) =================== (In reply to Rex Dieter from comment #16) > May be worth reviewing later commits/fixes to base boost-1.48.0 pkg up to: > http://pkgs.fedoraproject.org/cgit/boost.git/commit/ > ?id=dcde8a75575cb4c256ea982593c9710ac16b022c Done. Integrated all the missing patches. > 1 MUST fix missing -math dep in main metapackage and missing scriptlets > fixed in: > http://pkgs.fedoraproject.org/cgit/boost.git/commit/ > ?id=8aad81e67cbfd196e040449b0bf2cd854642c447 Done > 2. SHOULD consider using %{name} and %{?_isa} macros in place of > hard-coding boost148 in dependencies. imo, that simplifies things keeps > things consistent. for example, replace > Requires: boost148-chrono = %{version}-%{release} > with > Requires: %{name}-chrono%{?_isa} = %{version}-%{release} Done Did not check yet whether the fix for ppc64 works. ======================= -- 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

1 0

Jump to page:

2025

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

2010

2009

2008

2007

2006

package-review April 2015