FS#57800 - [libsavitar] Has 3 files common with community/pugixml

Attached to Project: Community Packages
Opened by ⁣ (eimis) - Monday, 12 March 2018, 03:35 GMT
Last edited by Jelle van der Waa (jelly) - Wednesday, 18 April 2018, 20:14 GMT
Task Type Bug Report
Category Packages
Status Closed
Assigned To Jelle van der Waa (jelly)
Architecture All
Severity Low
Priority Normal
Reported Version
Due in Version Undecided
Due Date Undecided
Percent Complete 100%
Votes 5
Private No

Details

Description:
# pacman -Syu
...
...
...
error: failed to commit transaction (conflicting files)
libsavitar: /usr/include/pugiconfig.hpp exists in filesystem
libsavitar: /usr/include/pugixml.hpp exists in filesystem
libsavitar: /usr/lib/cmake/pugixml/pugixml-config-release.cmake exists in filesystem
libsavitar: /usr/lib/cmake/pugixml/pugixml-config.cmake exists in filesystem
Errors occurred, no packages were upgraded.
Error installing repo packages.
$ pkgfile /usr/include/pugiconfig.hpp
community/pugixml

Additional info:
* package version(s) pugixml 1.8-1, libsavitar 2.6-1


Steps to reproduce:
# pacman -S pugixml libsavitar
This task depends upon

Closed by  Jelle van der Waa (jelly)
Wednesday, 18 April 2018, 20:14 GMT
Reason for closing:  Fixed
Additional comments about closing:  Fixed in 3.2.1
Comment by Jelle van der Waa (jelly) - Monday, 12 March 2018, 20:22 GMT
Great, libsavitar bundles pugiXML, this has to be unbundled somehow, see commit:
https://github.com/Ultimaker/libSavitar/commit/8ca8894a88e12e8f9353994d69ad43fe0dc5b32b
Comment by ⁣ (eimis) - Monday, 12 March 2018, 20:30 GMT
Then I lock my cura at 3.2.1-1 via the IgnorePkg for now.
Comment by Karim Lalani (jimmy00784) - Tuesday, 03 April 2018, 21:28 GMT
Are the common files needed once the package is installed? If not, perhaps removing them in PKGBUILD might work

I have the below modification in my local environment that seems to work for now although cura crashes at startup regardless:

package() {
cd "libSavitar-${pkgver}/build"
make DESTDIR="${pkgdir}" install
rm "${pkgdir}/usr/include/pugiconfig.hpp"
rm "${pkgdir}/usr/include/pugixml.hpp"
rm "${pkgdir}/usr/lib/cmake/pugixml/pugixml-config-release.cmake"
rm "${pkgdir}/usr/lib/cmake/pugixml/pugixml-config.cmake"
}
Comment by Eli Schwartz (eschwartz) - Monday, 09 April 2018, 06:40 GMT
https://github.com/Ultimaker/libSavitar/issues/7

What does this even do? It looks like it is broken by design...
Comment by Eli Schwartz (eschwartz) - Monday, 09 April 2018, 17:26 GMT
https://github.com/Ultimaker/libSavitar/pull/8 should fix this, consider cherry-picking if upstream does not fix and re-release soon.
Comment by Jelle van der Waa (jelly) - Wednesday, 18 April 2018, 20:14 GMT
Patch by eli, has been applied and the bug is resolved.

Loading...