[Bug 866265] Review Request: opentrep - C++ API for parsing travel-focused requests

https://bugzilla.redhat.com/show_bug.cgi?id=866265 --- Comment #41 from Fedora Update System --- opentrep-0.07.1-2.el7 has been pushed to the Fedora EPEL 7 stable repository. If problems still persist, please make note of it in this bug report. -- You are receiving this mail because: You are

[Bug 866265] Review Request: opentrep - C++ API for parsing travel-focused requests

https://bugzilla.redhat.com/show_bug.cgi?id=866265 Fedora Update System changed: What|Removed |Added Status|ON_QA |CLOSED Resolution|---

[Bug 866265] Review Request: opentrep - C++ API for parsing travel-focused requests

https://bugzilla.redhat.com/show_bug.cgi?id=866265 --- Comment #39 from Fedora Update System --- opentrep-0.07.1-2.fc29 has been pushed to the Fedora 29 testing repository. If problems still persist, please make note of it in this bug report. See

[Bug 866265] Review Request: opentrep - C++ API for parsing travel-focused requests

https://bugzilla.redhat.com/show_bug.cgi?id=866265 Fedora Update System changed: What|Removed |Added Status|MODIFIED|ON_QA --- Comment #38 from

[Bug 866265] Review Request: opentrep - C++ API for parsing travel-focused requests

https://bugzilla.redhat.com/show_bug.cgi?id=866265 --- Comment #37 from Fedora Update System --- opentrep-0.07.1-2.el7 has been submitted as an update to Fedora EPEL 7. https://bodhi.fedoraproject.org/updates/FEDORA-EPEL-2019-574d904e91 -- You are receiving this mail because: You are on the

[Bug 866265] Review Request: opentrep - C++ API for parsing travel-focused requests

https://bugzilla.redhat.com/show_bug.cgi?id=866265 Fedora Update System changed: What|Removed |Added Status|POST|MODIFIED --- Comment #36 from

[Bug 866265] Review Request: opentrep - C++ API for parsing travel-focused requests

https://bugzilla.redhat.com/show_bug.cgi?id=866265 --- Comment #35 from Gwyn Ciesla --- (fedscm-admin): The Pagure repository was created at https://src.fedoraproject.org/rpms/opentrep -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about

[Bug 866265] Review Request: opentrep - C++ API for parsing travel-focused requests

https://bugzilla.redhat.com/show_bug.cgi?id=866265 Robert-André Mauchin changed: What|Removed |Added Status|ASSIGNED|POST

[Bug 866265] Review Request: opentrep - C++ API for parsing travel-focused requests

https://bugzilla.redhat.com/show_bug.cgi?id=866265 --- Comment #33 from Denis Arnaud --- Successful build: https://koji.fedoraproject.org/koji/taskinfo?taskID=32875723 -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this

[Bug 866265] Review Request: opentrep - C++ API for parsing travel-focused requests

https://bugzilla.redhat.com/show_bug.cgi?id=866265 --- Comment #32 from Denis Arnaud --- (In reply to Robert-André Mauchin from comment #31) > - Please don't glob the major soname version to avoid unintentional soname > bump: > > %{_libdir}/lib%{name}.so.0* > > - The license file COPYING

[Bug 866265] Review Request: opentrep - C++ API for parsing travel-focused requests

https://bugzilla.redhat.com/show_bug.cgi?id=866265 Robert-André Mauchin changed: What|Removed |Added Status|NEW |ASSIGNED CC|

[Bug 866265] Review Request: opentrep - C++ API for parsing travel-focused requests

https://bugzilla.redhat.com/show_bug.cgi?id=866265 --- Comment #30 from Denis Arnaud --- Upstream update: Spec URL: http://denisarnaud.fedorapeople.org/opentrep/opentrep.spec (and https://github.com/fedorapackaging/fedorareviews/blob/trunk/reviews/opentrep/opentrep_866265/opentrep.spec) SRPM

[Bug 866265] Review Request: opentrep - C++ API for parsing travel-focused requests

https://bugzilla.redhat.com/show_bug.cgi?id=866265 Michael Schwendt changed: What|Removed |Added Blocks|201449 (FE-DEADREVIEW) | Referenced

[Bug 866265] Review Request: opentrep - C++ API for parsing travel-focused requests

https://bugzilla.redhat.com/show_bug.cgi?id=866265 gil cattaneo changed: What|Removed |Added CC||punto...@libero.it

[Bug 866265] Review Request: opentrep - C++ API for parsing travel-focused requests

https://bugzilla.redhat.com/show_bug.cgi?id=866265 --- Comment #29 from Denis Arnaud denis.arnaud_fed...@m4x.org --- Upstream update: Spec URL: http://denisarnaud.fedorapeople.org/opentrep/opentrep.spec SRPM URL: srpm info here Description:

[Bug 866265] Review Request: opentrep - C++ API for parsing travel-focused requests

https://bugzilla.redhat.com/show_bug.cgi?id=866265 --- Comment #28 from Michael Schwendt bugs.mich...@gmx.net --- Spotting issues or oddities often is the easy part during review. ;) Fixing them may be more difficult. -- You are receiving this mail because: You are on the CC list for the bug.

[Bug 866265] Review Request: opentrep - C++ API for parsing travel-focused requests

https://bugzilla.redhat.com/show_bug.cgi?id=866265 --- Comment #25 from Michael Schwendt bugs.mich...@gmx.net --- opentrep-python.x86_64: W: devel-file-in-non-devel-package /usr/lib64/python2.7/site-packages/libpyopentrep/libpyopentrep.so If you have any idea as how to remove that warning,

[Bug 866265] Review Request: opentrep - C++ API for parsing travel-focused requests

https://bugzilla.redhat.com/show_bug.cgi?id=866265 --- Comment #26 from Michael Schwendt bugs.mich...@gmx.net --- It's a limitation of the fedora-review tool, rpmlint precisely -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes

[Bug 866265] Review Request: opentrep - C++ API for parsing travel-focused requests

https://bugzilla.redhat.com/show_bug.cgi?id=866265 --- Comment #27 from Denis Arnaud denis.arnaud_fed...@m4x.org --- Many thanks, Michael, for your thorough review, as usual! (In reply to Michael Schwendt from comment #25) All are issues that sometimes are (re)introduced into packages also

[Bug 866265] Review Request: opentrep - C++ API for parsing travel-focused requests

https://bugzilla.redhat.com/show_bug.cgi?id=866265 --- Comment #23 from Denis Arnaud denis.arnaud_fed...@m4x.org --- Spec URL: http://denisarnaud.fedorapeople.org/opentrep/opentrep.spec SRPM URL: http://denisarnaud.fedorapeople.org/opentrep/opentrep-0.6.1-1.fc20.src.rpm Successful Koji build

[Bug 866265] Review Request: opentrep - C++ API for parsing travel-focused requests

https://bugzilla.redhat.com/show_bug.cgi?id=866265 --- Comment #24 from Denis Arnaud denis.arnaud_fed...@m4x.org --- (In reply to Michael Schwendt from comment #22) Many thanks, Michael! Details: pkgconfig deps are automatic for Fedora and EL6 Removed for EL = 5 Data sub-package:

[Bug 866265] Review Request: opentrep - C++ API for parsing travel-focused requests

https://bugzilla.redhat.com/show_bug.cgi?id=866265 --- Comment #22 from Michael Schwendt bugs.mich...@gmx.net --- %packagedevel Requires: pkgconfig pkgconfig deps are automatic for Fedora and EL6 %packagedata Summary:Referential data for the %{name} library

[Bug 866265] Review Request: opentrep - C++ API for parsing travel-focused requests

https://bugzilla.redhat.com/show_bug.cgi?id=866265 --- Comment #21 from Denis Arnaud denis.arnaud_fed...@m4x.org --- Spec URL: http://denisarnaud.fedorapeople.org/opentrep/opentrep.spec SRPM URL: http://denisarnaud.fedorapeople.org/opentrep/opentrep-0.6.0-1.fc20.src.rpm Successful Koji build

[Bug 866265] Review Request: opentrep - C++ API for parsing travel-focused requests

https://bugzilla.redhat.com/show_bug.cgi?id=866265 --- Comment #20 from Denis Arnaud denis.arnaud_fed...@m4x.org --- Spec URL: http://denisarnaud.fedorapeople.org/opentrep/opentrep.spec SRPM URL: http://denisarnaud.fedorapeople.org/opentrep/opentrep-0.6.0-1.fc20.src.rpm Successful Koji build

[Bug 866265] Review Request: opentrep - C++ API for parsing travel-focused requests

https://bugzilla.redhat.com/show_bug.cgi?id=866265 --- Comment #18 from Denis Arnaud denis.arnaud_fed...@m4x.org --- Spec URL: http://denisarnaud.fedorapeople.org/opentrep/opentrep.spec SRPM URL: http://denisarnaud.fedorapeople.org/opentrep/opentrep-0.6.0-1.fc20.src.rpm Succesful Koji build:

[Bug 866265] Review Request: opentrep - C++ API for parsing travel-focused requests

https://bugzilla.redhat.com/show_bug.cgi?id=866265 --- Comment #19 from Denis Arnaud denis.arnaud_fed...@m4x.org --- Spec URL: http://denisarnaud.fedorapeople.org/opentrep/opentrep.spec SRPM URL: http://denisarnaud.fedorapeople.org/opentrep/opentrep-0.6.0-1.fc20.src.rpm Successful Koji build

[Bug 866265] Review Request: opentrep - C++ API for parsing travel-focused requests

https://bugzilla.redhat.com/show_bug.cgi?id=866265 --- Comment #14 from Christopher Meng cicku...@gmail.com --- (In reply to Denis Arnaud from comment #13) (In reply to Christopher Meng from comment #12) ping. the package works, and it would make sense to release (approve) it as is. Of

[Bug 866265] Review Request: opentrep - C++ API for parsing travel-focused requests

https://bugzilla.redhat.com/show_bug.cgi?id=866265 Michael Schwendt bugs.mich...@gmx.net changed: What|Removed |Added Status|ASSIGNED|NEW

[Bug 866265] Review Request: opentrep - C++ API for parsing travel-focused requests

https://bugzilla.redhat.com/show_bug.cgi?id=866265 --- Comment #16 from Michael Schwendt bugs.mich...@gmx.net --- Re: comment 11 Multilib support with opentrep-config opentrep-devel.i686 and opentrep-devel.x86_64 conflict with eachother, because this script contains a hardcoded libdir

[Bug 866265] Review Request: opentrep - C++ API for parsing travel-focused requests

https://bugzilla.redhat.com/show_bug.cgi?id=866265 --- Comment #17 from Denis Arnaud denis.arnaud_fed...@m4x.org --- (In reply to Michael Schwendt from comment #16) Many thanks Michael, that is very helpful. I will have a look whenever I can. -- You are receiving this mail because: You are

[Bug 866265] Review Request: opentrep - C++ API for parsing travel-focused requests

https://bugzilla.redhat.com/show_bug.cgi?id=866265 --- Comment #12 from Christopher Meng cicku...@gmail.com --- ping. -- You are receiving this mail because: You are on the CC list for the bug. ___ package-review mailing list

[Bug 866265] Review Request: opentrep - C++ API for parsing travel-focused requests

https://bugzilla.redhat.com/show_bug.cgi?id=866265 --- Comment #13 from Denis Arnaud denis.arnaud_fed...@m4x.org --- (In reply to Christopher Meng from comment #12) ping. the package works, and it would make sense to release (approve) it as is. Of course, some more work would be needed to

[Bug 866265] Review Request: opentrep - C++ API for parsing travel-focused requests

https://bugzilla.redhat.com/show_bug.cgi?id=866265 Michael Schwendt bugs.mich...@gmx.net changed: What|Removed |Added Flags|needinfo?(bugs.michael@gmx. |

[Bug 866265] Review Request: opentrep - C++ API for parsing travel-focused requests

https://bugzilla.redhat.com/show_bug.cgi?id=866265 --- Comment #10 from Denis Arnaud denis.arnaud_fed...@m4x.org --- Many thanks, Michael, for that (piece of) review; it is already very valuable! I'll work on it asap, but probably not before next weekend (my day job keeps me quite busy these

[Bug 866265] Review Request: opentrep - C++ API for parsing travel-focused requests

https://bugzilla.redhat.com/show_bug.cgi?id=866265 --- Comment #11 from Denis Arnaud denis.arnaud_fed...@m4x.org --- Spec URL: http://denisarnaud.fedorapeople.org/opentrep/opentrep.spec SRPM URL: http://denisarnaud.fedorapeople.org/opentrep/opentrep-0.5.3-3.fc19.src.rpm Successful build on

[Bug 866265] Review Request: opentrep - C++ API for parsing travel-focused requests

https://bugzilla.redhat.com/show_bug.cgi?id=866265 --- Comment #8 from Denis Arnaud denis.arnaud_fed...@m4x.org --- Spec URL: http://denisarnaud.fedorapeople.org/opentrep/opentrep.spec SRPM URL: http://denisarnaud.fedorapeople.org/opentrep/opentrep-0.5.3-2.fc19.src.rpm Successful build on Koji

[Bug 866265] Review Request: opentrep - C++ API for parsing travel-focused requests

https://bugzilla.redhat.com/show_bug.cgi?id=866265 Christopher Meng cicku...@gmail.com changed: What|Removed |Added CC|

[Bug 866265] Review Request: opentrep - C++ API for parsing travel-focused requests

https://bugzilla.redhat.com/show_bug.cgi?id=866265 Michael Schwendt mschwe...@gmail.com changed: What|Removed |Added Status|NEW |ASSIGNED

[Bug 866265] Review Request: opentrep - C++ API for parsing travel-focused requests

Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=866265 --- Comment #7 from Denis Arnaud denis.arnaud_fed...@m4x.org --- Spec URL: http://denisarnaud.fedorapeople.org/opentrep/opentrep.spec SRPM URL: http://denisarnaud.fedorapeople.org/opentrep/opentrep-0.5.3-1.fc18.src.rpm -- You

[Bug 866265] Review Request: opentrep - C++ API for parsing travel-focused requests

Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=866265 --- Comment #6 from Denis Arnaud denis.arnaud_fed...@m4x.org --- Spec URL: http://denisarnaud.fedorapeople.org/opentrep/opentrep.spec SRPM URL: http://denisarnaud.fedorapeople.org/opentrep/opentrep-0.5.2-1.fc18.src.rpm -- You

[Bug 866265] Review Request: opentrep - C++ API for parsing travel-focused requests

Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=866265 --- Comment #4 from Michael Schwendt mschwe...@gmail.com --- %dir %{_datadir}/%{name} %dir %{_datadir}/%{name}/data %dir %{_datadir}/%{name}/data/por %{_datadir}/%{name}/data/por/ori_por_public.csv is simple enough. [There

[Bug 866265] Review Request: opentrep - C++ API for parsing travel-focused requests

Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=866265 --- Comment #5 from Denis Arnaud denis.arnaud_fed...@m4x.org --- Thanks Michael! The package is still waiting for an approval. So, do not hesitate to come forward :) -- You are receiving this mail because: You are on the CC list

[Bug 866265] Review Request: opentrep - C++ API for parsing travel-focused requests

https://bugzilla.redhat.com/show_bug.cgi?id=866265 --- Comment #3 from Denis Arnaud denis.arnaud_fed...@m4x.org --- [For later reference, I just add the corresponding URLs of the packaging guidelines] Spec URL: http://denisarnaud.fedorapeople.org/opentrep/opentrep.spec SRPM URL:

[Bug 866265] Review Request: opentrep - C++ API for parsing travel-focused requests

https://bugzilla.redhat.com/show_bug.cgi?id=866265 --- Comment #2 from Denis Arnaud denis.arnaud_fed...@m4x.org --- Thanks Volker for the review. I will have a look at it shortly and publish the amended versions of the package. -- You are receiving this mail because: You are on the CC list for

[Bug 866265] Review Request: opentrep - C++ API for parsing travel-focused requests

https://bugzilla.redhat.com/show_bug.cgi?id=866265 Volker Fröhlich volke...@gmx.at changed: What|Removed |Added CC||volke...@gmx.at ---