[Bug 972431] Review Request: airtsp - C++ Simulated Airline Travel Solution Provider Library
https://bugzilla.redhat.com/show_bug.cgi?id=972431 --- Comment #14 from Christopher Meng cicku...@gmail.com --- I think all issues are fixed, time to approve. -- 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 972431] Review Request: airtsp - C++ Simulated Airline Travel Solution Provider Library
https://bugzilla.redhat.com/show_bug.cgi?id=972431 --- Comment #15 from Ralf Corsepius rc040...@freenet.de --- (In reply to Christopher Meng from comment #14) I think all issues are fixed, time to approve. There still are arch/multilib-related issues. Try building the srpm for i386 and x86_64, unpackage the resulting binary rpms into separate directories and compare them. Here is one example (There are several more): diff -Naur i386/usr/share/airtsp/CMake/airtsp-config.cmake x86_64/usr/share/airtsp/CMake/airtsp-config.cmake --- i386/usr/share/airtsp/CMake/airtsp-config.cmake 2013-11-04 06:16:02.0 +0100 +++ x86_64/usr/share/airtsp/CMake/airtsp-config.cmake 2013-11-04 06:01:34.0 +0100 @@ -11,7 +11,7 @@ set (AIRTSP_VERSION 1.01.1) set (AIRTSP_BINARY_DIRS /usr/bin) set (AIRTSP_INCLUDE_DIRS /usr/include) -set (AIRTSP_LIBRARY_DIRS /usr/lib) +set (AIRTSP_LIBRARY_DIRS /usr/lib64) # Library dependencies for AirTSP (contains definitions for the AirTSP # IMPORTED targets) Additionally, the doc-package is having doxygen timestamping issues, which causes the doc package to be built non-deterministically: # diff -Naur i386/usr/share/doc/airtsp-doc/html/AirlineScheduleTestSuite_8cpp.html x86_64/usr/share/doc/airtsp-doc/html/AirlineScheduleTestSuite_8cpp.html --- i386/usr/share/doc/airtsp-doc/html/AirlineScheduleTestSuite_8cpp.html 2013-11-04 06:16:05.0 +0100 +++ x86_64/usr/share/doc/airtsp-doc/html/AirlineScheduleTestSuite_8cpp.html 2013-11-04 06:01:36.0 +0100 @@ -130,7 +130,7 @@ /div!-- contents -- /div!-- doc-content -- - li class=footerGenerated on Mon Nov 4 2013 06:16:05 for AirTSP by + li class=footerGenerated on Mon Nov 4 2013 06:01:36 for AirTSP by a href=http://www.doxygen.org/index.html; img class=footer src=doxygen.png alt=doxygen//a 1.8.5 -- 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 972431] Review Request: airtsp - C++ Simulated Airline Travel Solution Provider Library
https://bugzilla.redhat.com/show_bug.cgi?id=972431 Björn besser82 Esser bjoern.es...@gmail.com changed: What|Removed |Added Flags|needinfo?(bjoern.esser@gmai | |l.com) | --- Comment #12 from Björn besser82 Esser bjoern.es...@gmail.com --- (In reply to Christopher Meng from comment #10) You should check this unfinished one. You dont't say. :D I'm on it, but since this is a RENAME-request it doesn't hurry that much, because it's already available in the repos. (In reply to Christopher Meng from comment #11) Denis, do you want to support EL5? Since the package, to be renamed, is build for el5, too, the RENAME-request should cover el5 as well. -- 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 972431] Review Request: airtsp - C++ Simulated Airline Travel Solution Provider Library
https://bugzilla.redhat.com/show_bug.cgi?id=972431 --- Comment #13 from Denis Arnaud denis.arnaud_fed...@m4x.org --- (In reply to Christopher Meng from comment #11) Denis, do you want to support EL5? Yes, I do -- 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 972431] Review Request: airtsp - C++ Simulated Airline Travel Solution Provider Library
https://bugzilla.redhat.com/show_bug.cgi?id=972431 Christopher Meng cicku...@gmail.com changed: What|Removed |Added Flags||needinfo?(bjoern.esser@gmai ||l.com) --- Comment #10 from Christopher Meng cicku...@gmail.com --- You should check this unfinished one. -- 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 972431] Review Request: airtsp - C++ Simulated Airline Travel Solution Provider Library
https://bugzilla.redhat.com/show_bug.cgi?id=972431 --- Comment #11 from Christopher Meng cicku...@gmail.com --- Denis, do you want to support EL5? -- 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 972431] Review Request: airtsp - C++ Simulated Airline Travel Solution Provider Library
https://bugzilla.redhat.com/show_bug.cgi?id=972431 Ralf Corsepius rc040...@freenet.de changed: What|Removed |Added CC||rc040...@freenet.de --- Comment #8 from Ralf Corsepius rc040...@freenet.de --- MUSTFIX: * /usr/bin/airtsp-config is not multilib-capable -- 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=pr3Vjt33NMa=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 972431] Review Request: airtsp - C++ Simulated Airline Travel Solution Provider Library
https://bugzilla.redhat.com/show_bug.cgi?id=972431 --- Comment #9 from Denis Arnaud denis.arnaud_fed...@m4x.org --- Spec URL: http://denisarnaud.fedorapeople.org/sim/airtsp/airtsp.spec SRPM URL: http://denisarnaud.fedorapeople.org/sim/airtsp/airtsp-1.01.1-1.fc19.src.rpm Successful build on Rawhide: http://koji.fedoraproject.org/koji/taskinfo?taskID=5809903 -- 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=wAX1mJTgafa=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 972431] Review Request: airtsp - C++ Simulated Airline Travel Solution Provider Library
https://bugzilla.redhat.com/show_bug.cgi?id=972431 --- Comment #7 from Denis Arnaud denis.arnaud_fed...@m4x.org --- Spec URL: http://denisarnaud.fedorapeople.org/sim/airtsp/airtsp.spec SRPM URL: http://denisarnaud.fedorapeople.org/sim/airtsp/airtsp-1.01.0-3.fc19.src.rpm Successful build on Rawhide: http://koji.fedoraproject.org/koji/taskinfo?taskID=5805038 -- 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=EfxHBkemVga=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 972431] Review Request: airtsp - C++ Simulated Airline Travel Solution Provider Library
https://bugzilla.redhat.com/show_bug.cgi?id=972431
Christopher Meng cicku...@gmail.com changed:
What|Removed |Added
CC||cicku...@gmail.com
--- Comment #6 from Christopher Meng cicku...@gmail.com ---
Remove rm -rf $RPM_BUILD_ROOT
Remove %clean
rm -rf $RPM_BUILD_ROOT
Remove %defattr(-,root,root,-)
Above only for EL5+, if you want to package it for EL5, ignore them.
BUT,
Reflected by Fedora 20 Change, please modify your docdir to %{_pkgdocdir}.
--
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=2WNzekfBqqa=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 972431] Review Request: airtsp - C++ Simulated Airline Travel Solution Provider Library
https://bugzilla.redhat.com/show_bug.cgi?id=972431 --- Comment #2 from Björn Esser bjoern.es...@gmail.com --- Created attachment 760156 -- https://bugzilla.redhat.com/attachment.cgi?id=760156action=edit epstopdf and graphviz are missing for building the auto-docs The autodocs are not build properly caused by missing dot (graphviz), epstopdf (and possibly some more). Please fix and I'll start a new run. -- 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=XiBfzj5YhWa=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 972431] Review Request: airtsp - C++ Simulated Airline Travel Solution Provider Library
https://bugzilla.redhat.com/show_bug.cgi?id=972431 --- Comment #3 from Denis Arnaud denis.arnaud_fed...@m4x.org --- (In reply to Björn Esser from comment #2) Created attachment 760156 [details] epstopdf and graphviz are missing for building the auto-docs Done Spec URL: http://denisarnaud.fedorapeople.org/sim/airtsp/airtsp.spec SRPM URL: http://denisarnaud.fedorapeople.org/sim/airtsp/airtsp-1.01.0-2.fc17.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=xUiMgIsxq1a=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 972431] Review Request: airtsp - C++ Simulated Airline Travel Solution Provider Library
https://bugzilla.redhat.com/show_bug.cgi?id=972431 --- Comment #4 from Björn Esser bjoern.es...@gmail.com --- Now it's FTBFS on RAWHIDE: Error: No Package found for texlive-utils -- 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=CwD2vWJh4za=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 972431] Review Request: airtsp - C++ Simulated Airline Travel Solution Provider Library
https://bugzilla.redhat.com/show_bug.cgi?id=972431 --- Comment #5 from Denis Arnaud denis.arnaud_fed...@m4x.org --- (In reply to Björn Esser from comment #4) Now it's FTBFS on RAWHIDE: Error: No Package found for texlive-utils Sorry for that (I tested on Fedora 17, as it is the only one available I have under hand right now). The files have been fixed and overwritten (without bumping the version, as it was not building successfully): Spec URL: http://denisarnaud.fedorapeople.org/sim/airtsp/airtsp.spec SRPM URL: http://denisarnaud.fedorapeople.org/sim/airtsp/airtsp-1.01.0-2.fc17.src.rpm The build on Rawhide is successful: http://koji.fedoraproject.org/koji/taskinfo?taskID=5496327 -- 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=QIw7UyLZNMa=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 972431] Review Request: airtsp - C++ Simulated Airline Travel Solution Provider Library
https://bugzilla.redhat.com/show_bug.cgi?id=972431 --- Comment #1 from Denis Arnaud denis.arnaud_fed...@m4x.org --- Note that this package has been renamed from AirSched. See, for references: * AirSched review request: http://bugzilla.redhat.com/show_bug.cgi?id=732205 * http://fedoraproject.org/wiki/Packaging:Guidelines#Renaming.2FReplacing_Existing_Packages * http://fedoraproject.org/wiki/Package_Renaming_Process -- 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=Y2Q5jBaEHca=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 972431] Review Request: airtsp - C++ Simulated Airline Travel Solution Provider Library
https://bugzilla.redhat.com/show_bug.cgi?id=972431 Björn Esser bjoern.es...@gmail.com changed: What|Removed |Added Status|NEW |ASSIGNED CC||bjoern.es...@gmail.com Assignee|nob...@fedoraproject.org|bjoern.es...@gmail.com Flags||fedora-review? -- 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=Ra0lJLPOpsa=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
