3786 2013-01-23 21:31:43 +0000 RPM packaging fails for pkgconfig files with '-' in version field 2017-07-24 15:12:58 +0000 1 1 7 Build System, Metadata & Runtime OE-Core core unspecified x86 Multiple RESOLVED OBSOLETE Low minor 2.4 M3 1 philip alex.kanavin mark.hatle meta.mr.watcher meta.watcher n3npq richard.purdie sgw --- --- Don't know oldest_to_newest 29113 0 philip 2013-01-23 21:31:43 +0000 When building uhd (from meta-oe) AND enabling rpm packaging the rpm creation fails with an odd error: .52.0 libstdc++.so.6(GLIBCXX_3.4) libc.so.6 libc.so.6(GLIBC_2.4) libc.so.6(GLIBC_2.9) libc.so.6(GLIBC_2.7) libc.so.6(GLIBC_2.8) libgcc_s.so.1(GCC_4.0.0) libtinfo.so.5 libncurses.so.5 | Processing files: uhd-dbg-3.5.0-r3.0.armv7a_vfp_neon | Finding Provides: /home/balister/src/oe-core/build/tmp-eglibc/work/armv7a-vfp-neon-oe-linux-gnueabi/uhd/1_3.5.0-r3.0/uhd.provides | Finding Requires: /home/balister/src/oe-core/build/tmp-eglibc/work/armv7a-vfp-neon-oe-linux-gnueabi/uhd/1_3.5.0-r3.0/uhd.requires | Processing files: uhd-dev-3.5.0-r3.0.armv7a_vfp_neon | Finding Provides: /home/balister/src/oe-core/build/tmp-eglibc/work/armv7a-vfp-neon-oe-linux-gnueabi/uhd/1_3.5.0-r3.0/uhd.provides | error: line 719: Illegal char '-' in EVR: | | error: Failed to find Provides: | | | RPM build errors: | line 719: Illegal char '-' in EVR: | | Failed to find Provides: | DEBUG: Python function do_package_rpm finished The recipe does find with opkg. Mark Hatle looked at the uhd rpm files and suggested this: do_install_append() { rm -f ${D}/usr/lib/pkgconfig/uhd.pc } This "solves" the problem, but something is not right. Mark asked me to create this bug. This should be reproducible with just oe-core + meta-oe. 29114 1 mark.hatle 2013-01-23 22:02:03 +0000 I have successfully reproduced the problem. Reproducer: download oe-core/bitbake download meta-openembedded configure a new build edit conf/bblayers.conf -- add meta-openembedded/meta-oe bitbake ncurses bitbake uhd (uhd has a minor bug in it's dependencies, it requires ncurses but does not yet state it.) Result of the above matches Philip's comments. 29115 2 mark.hatle 2013-01-23 22:20:34 +0000 Contents of packages-split/uhd-dev/usr/lib/pkgconfig/uhd.pc: prefix=/usr exec_prefix=${prefix} libdir=${exec_prefix}/lib includedir=${prefix}/include Name: UHD Description: Ettus Research - USRP Hardware Driver URL: http://code.ettus.com/redmine/ettus/projects/uhd/wiki Version: 003.005.000-0-g5cb9779d Requires: Requires.private: Conflicts: Cflags: -I${includedir} Libs: -L${libdir} -luhd Libs.private: --- This causes the pkgconfig dependency code to pull out a dep on "Version" 003.005.000-0-g5cb9779d of the "pkgconfig(uhd)". RPM is assuming the versioning should be handled by standard rules, which are "version-release", where "-" is not supported in either version or release. Thus three '-' means it can't figure out which is version and which is release, thus the error is generated. Still don't have a solution, but I can explain the fault at least. 29149 3 n3npq 2013-01-24 19:59:09 +0000 Probably the most general/simplest fix would be to add ... | tr -- - _ (or equivalent) to the script that extracts the version from the *.pc file. The file that needs changing is pkgconfigdeps.sh, the version is being extracted by these 2 lines $pkgconfig --print-provides "$filename" 2> /dev/null | while read n r v ; do ... $pkgconfig --print-requires "$filename" 2> /dev/null | while read n r v ; do and you would need to split out the EVR field and change all '-' dash characters to '_' underscore characters in the EVR field. 29150 4 philip 2013-01-24 20:05:30 +0000 I received this in the email also: --- host/cmake/Modules/UHDVersion.cmake 2013-01-24 16:11:20.956506736 +0100 +++ host.b/cmake/Modules/UHDVersion.cmake 2013-01-24 17:11:22.966407775 +0100 @@ -69,7 +69,7 @@ ENDIF() ######################################################################## IF(UHD_RELEASE_MODE STREQUAL "release") - SET(UHD_VERSION "${UHD_VERSION_MAJOR}.${UHD_VERSION_MINOR}.${UHD_VERSION_PATCH}-${UHD_GIT_HASH}") + SET(UHD_VERSION "${UHD_VERSION_MAJOR}.${UHD_VERSION_MINOR}.${UHD_VERSION_PATCH}+${UHD_GIT_HASH}") ELSE() - SET(UHD_VERSION "${UHD_VERSION_MAJOR}.${UHD_VERSION_MINOR}.${UHD_VERSION_PATCH}-${UHD_GIT_COUNT}-${UHD_GIT_HASH}") + SET(UHD_VERSION "${UHD_VERSION_MAJOR}.${UHD_VERSION_MINOR}.${UHD_VERSION_PATCH}+${UHD_GIT_COUNT}+${UHD_GIT_HASH}") ENDIF() This patches uhd to avoid the problem. I'll discuss with the UHD maintainer. 29151 5 n3npq 2013-01-24 20:07:58 +0000 Actually all the splitting is done: just wrap a back ticked sed/tr onto the $v field in the two places that look like echo "pkgconfig($n) $r $v" 29152 6 n3npq 2013-01-24 20:09:03 +0000 Yes changing the Version: string also Just Works (but doesn't solve the problem generally and is out-of-scope as an RPM "fix") 30051 7 mark.hatle 2013-02-22 20:01:26 +0000 The original problem was resolved by a change to UHD. The underlying issue still needs to be resolved via a patch to the pkgconfig scanning code in rpmdeps. 32800 8 mark.hatle 2013-04-30 23:19:27 +0000 We will work on this for the 1.5 release. The workaround should be good enough for 1.4, and the issue is fairly rare. 34622 9 mark.hatle 2013-07-16 18:01:55 +0000 Not deployment, bug in RPM integration. 42044 10 mark.hatle 2014-04-01 14:49:17 +0000 We've had no more reports of this.. but it should be fixed. Moving it forward to 1.7. 75195 11 richard.purdie 2017-07-20 15:27:40 +0000 Alex: Could you check if we have this issue with rpmdeps in rpm v4 please? 75255 12 alex.kanavin 2017-07-24 15:12:58 +0000 I have baked a version with two dashes into sqlite3.pc.in file, and rpm4 had no problem with that (maybe it doesn't look into .pc files at all anymore?). I'd say RESOLVED/OBSOLETE.