[Bug 957346] Review Request: mingw-physfs - MinGW compiled physfs library to provide abstract access to various archives
https://bugzilla.redhat.com/show_bug.cgi?id=957346 Fedora Update System upda...@fedoraproject.org changed: What|Removed |Added Status|ON_QA |CLOSED Fixed In Version||mingw-physfs-2.0.3-4.fc19 Resolution|--- |ERRATA Last Closed||2014-05-29 19:26:27 --- Comment #13 from Fedora Update System upda...@fedoraproject.org --- mingw-physfs-2.0.3-4.fc19 has been pushed to the Fedora 19 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 957346] Review Request: mingw-physfs - MinGW compiled physfs library to provide abstract access to various archives
https://bugzilla.redhat.com/show_bug.cgi?id=957346 Fedora Update System upda...@fedoraproject.org changed: What|Removed |Added Fixed In Version|mingw-physfs-2.0.3-4.fc19 |mingw-physfs-2.0.3-4.fc20 --- Comment #14 from Fedora Update System upda...@fedoraproject.org --- mingw-physfs-2.0.3-4.fc20 has been pushed to the Fedora 20 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 957346] Review Request: mingw-physfs - MinGW compiled physfs library to provide abstract access to various archives
https://bugzilla.redhat.com/show_bug.cgi?id=957346 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 957346] Review Request: mingw-physfs - MinGW compiled physfs library to provide abstract access to various archives
https://bugzilla.redhat.com/show_bug.cgi?id=957346 --- Comment #10 from Fedora Update System upda...@fedoraproject.org --- mingw-physfs-2.0.3-4.fc19 has been submitted as an update for Fedora 19. https://admin.fedoraproject.org/updates/mingw-physfs-2.0.3-4.fc19 -- 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 957346] Review Request: mingw-physfs - MinGW compiled physfs library to provide abstract access to various archives
https://bugzilla.redhat.com/show_bug.cgi?id=957346 --- Comment #11 from Fedora Update System upda...@fedoraproject.org --- mingw-physfs-2.0.3-4.fc20 has been submitted as an update for Fedora 20. https://admin.fedoraproject.org/updates/mingw-physfs-2.0.3-4.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 957346] Review Request: mingw-physfs - MinGW compiled physfs library to provide abstract access to various archives
https://bugzilla.redhat.com/show_bug.cgi?id=957346 MinGW Maintenance Account fedora-mi...@lists.fedoraproject.org changed: What|Removed |Added CC|fedora-mingw@lists.fedorapr | |oject.org | Fedora Update System upda...@fedoraproject.org changed: What|Removed |Added Status|MODIFIED|ON_QA --- Comment #12 from Fedora Update System upda...@fedoraproject.org --- Package mingw-physfs-2.0.3-4.fc20: * should fix your issue, * was pushed to the Fedora 20 testing repository, * should be available at your local mirror within two days. Update it with: # su -c 'yum update --enablerepo=updates-testing mingw-physfs-2.0.3-4.fc20' as soon as you are able to. Please go to the following url: https://admin.fedoraproject.org/updates/FEDORA-2014-6368/mingw-physfs-2.0.3-4.fc20 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 957346] Review Request: mingw-physfs - MinGW compiled physfs library to provide abstract access to various archives
https://bugzilla.redhat.com/show_bug.cgi?id=957346 Marcel Wysocki m...@satgnu.net changed: What|Removed |Added Flags||fedora-cvs? --- Comment #8 from Marcel Wysocki m...@satgnu.net --- New Package SCM Request === Package Name: mingw-physfs Short Description: MinGW compiled physfs library to provide abstract access to various archives Owners: maci epienbro Branches: f19 f20 epel7 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 957346] Review Request: mingw-physfs - MinGW compiled physfs library to provide abstract access to various archives
https://bugzilla.redhat.com/show_bug.cgi?id=957346 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 957346] Review Request: mingw-physfs - MinGW compiled physfs library to provide abstract access to various archives
https://bugzilla.redhat.com/show_bug.cgi?id=957346 --- Comment #9 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 957346] Review Request: mingw-physfs - MinGW compiled physfs library to provide abstract access to various archives
https://bugzilla.redhat.com/show_bug.cgi?id=957346 Erik van Pienbroek erik-fed...@vanpienbroek.nl changed: What|Removed |Added Flags|fedora-review? |fedora-review+ --- Comment #7 from Erik van Pienbroek erik-fed...@vanpienbroek.nl --- My apologies on the long delay. I forgot about this review ticket. All issues are resolved now: == The package mingw-physfs is APPROVED by epienbro == -- 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 957346] Review Request: mingw-physfs - MinGW compiled physfs library to provide abstract access to various archives
https://bugzilla.redhat.com/show_bug.cgi?id=957346 --- Comment #6 from Marcel Wysocki m...@satgnu.net --- anything still blocking this? -- 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 957346] Review Request: mingw-physfs - MinGW compiled physfs library to provide abstract access to various archives
https://bugzilla.redhat.com/show_bug.cgi?id=957346 Marcel Wysocki m...@satgnu.net changed: What|Removed |Added Depends On|971115 | --- Comment #5 from Marcel Wysocki m...@satgnu.net --- Update: %changelog * Mon Oct 14 2013 maci m...@satgnu.net - 2.0.3-4 - remove patch - build without 7zip support, its unlikely mingw-lzma-sdk457 will ever work. Can be re-enabled when xz-devel support is implemented http://icculus.org/pipermail/physfs/2010-December/000971.html Spec URL: https://raw.github.com/maci0/rpmbuild/master/SPECS/mingw-physfs.spec SRPM URL: https://raw.github.com/maci0/rpmbuild/master/SRPMS/mingw-physfs-2.0.3-4.fc19.src.rpm Koji URL: http://koji.fedoraproject.org/koji/taskinfo?taskID=6057794 Referenced Bugs: https://bugzilla.redhat.com/show_bug.cgi?id=971115 [Bug 971115] Review Request: mingw-lzma-sdk457 - SDK for lzma compression -- 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 957346] Review Request: mingw-physfs - MinGW compiled physfs library to provide abstract access to various archives
https://bugzilla.redhat.com/show_bug.cgi?id=957346 --- Comment #4 from Marcel Wysocki m...@satgnu.net --- Update: * Thu Jun 06 2013 Marcel Wysocki m...@satgnu.net - 2.0.3-3 - rebuilt for mingw-lzma-sdk457 package Spec URL: http://maci.satgnu.net/rpmbuild/SPECS/mingw-physfs.spec SRPM URL: http://maci.satgnu.net/rpmbuild/SRPMS/mingw-physfs-2.0.3-3.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=WzR4yN7sDVa=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 957346] Review Request: mingw-physfs - MinGW compiled physfs library to provide abstract access to various archives
https://bugzilla.redhat.com/show_bug.cgi?id=957346 Marcel Wysocki m...@satgnu.net changed: What|Removed |Added Depends On||971115 -- 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=2LyWtM8DBpa=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 957346] Review Request: mingw-physfs - MinGW compiled physfs library to provide abstract access to various archives
Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=957346 --- Comment #3 from Erik van Pienbroek erik-fed...@vanpienbroek.nl --- It looks like physfs contains a bundled copy of lzma. According to the packaging guidelines it isn't allowed to use bundled copies of external libraries. Now there are two options on how to continue: 1. Strip the lzma pieces from physfs (thus also removing lzma support in the library itself) 2. Create a dedicated mingw-lzma package and have this package use it A choice from one of these options needs to be made before this package can be approved -- 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=6qBll7oESka=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 957346] Review Request: mingw-physfs - MinGW compiled physfs library to provide abstract access to various archives
Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=957346 --- Comment #2 from Marcel Wysocki m...@satgnu.net --- Fixed most issues: Spec URL: http://maci.satgnu.net/rpmbuild/SPECS/mingw-physfs.spec SRPM URL: http://maci.satgnu.net/rpmbuild/SRPMS/mingw-physfs-2.0.3-2.fc19.src.rpm The patch is from the fedora physfs package and enables physfs to use a system provided lzma sdk instead of the one bundled with physfs (violates packaging guidelines). there exists no mingw package for the lzma sdk tho. How should I proceed ? Im not sure the lzma sdk package would even compile for mingw, its quite outdated as well. -- 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=CYO5m36ZyAa=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 957346] Review Request: mingw-physfs - MinGW compiled physfs library to provide abstract access to various archives
Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=957346
Erik van Pienbroek erik-fed...@vanpienbroek.nl changed:
What|Removed |Added
Status|NEW |ASSIGNED
CC||erik-fed...@vanpienbroek.nl
Assignee|nob...@fedoraproject.org|erik-fed...@vanpienbroek.nl
Flags||fedora-review?
--- Comment #1 from Erik van Pienbroek erik-fed...@vanpienbroek.nl ---
$ rpmlint mingw-physfs.spec
mingw-physfs.spec:12: W: mixed-use-of-spaces-and-tabs (spaces: line 3, tab:
line 12)
0 packages and 1 specfiles checked; 0 errors, 1 warnings.
$ rpmlint mingw-physfs-2.0.3-1.fc19.src.rpm
mingw-physfs.src: W: spelling-error %description -l en_US filesystem - file
system, file-system, systemically
mingw-physfs.src: W: spelling-error %description -l en_US untrusted -
entrusted, trusted, encrusted
mingw-physfs.src: E: description-line-too-long C MinGW compiled PhysicsFS, a
library to provide abstract access to various archives. It is
mingw-physfs.src:12: W: mixed-use-of-spaces-and-tabs (spaces: line 3, tab: line
12)
1 packages and 0 specfiles checked; 1 errors, 3 warnings.
$ rpmlint mingw32-physfs-2.0.3-1.fc19.noarch.rpm
mingw32-physfs-static-2.0.3-1.fc19.noarch.rpm
mingw64-physfs-2.0.3-1.fc19.noarch.rpm
mingw64-physfs-static-2.0.3-1.fc19.noarch.rpm
mingw32-physfs.noarch: W: spelling-error %description -l en_US filesystem -
file system, file-system, systemically
mingw32-physfs.noarch: W: spelling-error %description -l en_US untrusted -
entrusted, trusted, encrusted
mingw32-physfs.noarch: E: description-line-too-long C MinGW compiled PhysicsFS,
a library to provide abstract access to various archives. It is
mingw32-physfs.noarch: W: no-documentation
mingw32-physfs-static.noarch: W: no-documentation
mingw64-physfs.noarch: W: spelling-error %description -l en_US filesystem -
file system, file-system, systemically
mingw64-physfs.noarch: W: spelling-error %description -l en_US untrusted -
entrusted, trusted, encrusted
mingw64-physfs.noarch: E: description-line-too-long C MinGW compiled PhysicsFS,
a library to provide abstract access to various archives. It is
mingw64-physfs.noarch: W: no-documentation
mingw64-physfs-static.noarch: W: no-documentation
4 packages and 0 specfiles checked; 2 errors, 8 warnings.
$ rpm --query --requires mingw32-physfs
rpmlib(PayloadFilesHavePrefix) = 4.0-1
rpmlib(CompressedFileNames) = 3.0.4-1
mingw32(advapi32.dll)
mingw32-crt
mingw32-filesystem = 83
mingw32(kernel32.dll)
mingw32(libgcc_s_sjlj-1.dll)
mingw32(msvcrt.dll)
mingw32(user32.dll)
mingw32(zlib1.dll)
rpmlib(PayloadIsXz) = 5.2-1
$ rpm --query --requires mingw64-physfs
rpmlib(PayloadFilesHavePrefix) = 4.0-1
rpmlib(CompressedFileNames) = 3.0.4-1
mingw64(advapi32.dll)
mingw64-crt
mingw64-filesystem = 83
mingw64(kernel32.dll)
mingw64(msvcrt.dll)
mingw64(user32.dll)
mingw64(zlib1.dll)
rpmlib(PayloadIsXz) = 5.2-1
$ rpm --query --requires mingw32-physfs-static
mingw32-physfs = 2.0.3-1.fc19
rpmlib(PayloadFilesHavePrefix) = 4.0-1
rpmlib(CompressedFileNames) = 3.0.4-1
rpmlib(PayloadIsXz) = 5.2-1
$ rpm --query --requires mingw64-physfs-static
mingw64-physfs = 2.0.3-1.fc19
rpmlib(PayloadFilesHavePrefix) = 4.0-1
rpmlib(CompressedFileNames) = 3.0.4-1
rpmlib(PayloadIsXz) = 5.2-1
$ rpm --query --provides mingw32-physfs
mingw32-physfs = 2.0.3-1.fc19
mingw32(libphysfs.dll)
$ rpm --query --provides mingw64-physfs
mingw64-physfs = 2.0.3-1.fc19
mingw64(libphysfs.dll)
$ rpm --query --provides mingw32-physfs-static
mingw32-physfs-static = 2.0.3-1.fc19
$ rpm --query --provides mingw64-physfs-static
mingw64-physfs-static = 2.0.3-1.fc19
$ wget --quiet http://www.icculus.org/physfs/downloads/physfs-2.0.3.tar.bz2 -O
- | md5sum
c2c727a8a8deb623b521b52d0080f613 -
$ md5sum physfs-2.0.3.tar.bz2
c2c727a8a8deb623b521b52d0080f613 physfs-2.0.3.tar.bz2
+ OK
! Needs to be looked into
/ Not applicable
[!] Compliant with generic Fedora Packaging Guidelines
[+] Source package name is prefixed with 'mingw-'
[+] Spec file starts with %{?mingw_package_header}
[+] BuildRequires: mingw32-filesystem = 95 is in the .spec file
[+] BuildRequires: mingw64-filesystem = 95 is in the .spec file
[+] Spec file contains %package sections for both mingw32 and mingw64 packages
[+] Binary mingw32 and mingw64 packages are noarch
[+] Spec file contains %{?mingw_debug_package} after the %description section
[+] Uses one of the macros %mingw_configure, %mingw_cmake, or %mingw_cmake_kde4
to configure the package
[+] Uses the macro %mingw_make to build the package
[+] Uses the macro %mingw_make to install the package
[+] If package contains translations, the %mingw_find_lang macro must be used
[+] No binary package named mingw-$pkgname is generated
[+] Libtool .la files are not bundled
[+] .def files are not bundled
[+] Man pages which duplicate
