Bug 2045568

Summary:	hunspell-1.7.0-13 causes image compose failures due to missing ln
Product:	[Fedora] Fedora	Reporter:	Kevin Fenzi <kevin>
Component:	hunspell	Assignee:	Jens Petersen <petersen>
Status:	CLOSED RAWHIDE	QA Contact:	Fedora Extras Quality Assurance <extras-qa>
Severity:	high	Docs Contact:
Priority:	unspecified
Version:	rawhide	CC:	caillon+fedoraproject, caolanm, gnome-sig, mclasen, mhroncok, pnemade, rhughes, rstrode, sandmann, vishalvijayraghavan
Target Milestone:	---
Target Release:	---
Hardware:	Unspecified
OS:	Unspecified
Whiteboard:
Fixed In Version:	hunspell-1.7.0-15.fc36	Doc Type:	If docs needed, set a value
Doc Text:		Story Points:	---
Clone Of:		Environment:
Last Closed:	2022-01-31 09:28:09 UTC	Type:	Bug
Regression:	---	Mount Type:	---
Documentation:	---	CRM:
Verified Versions:		Category:	---
oVirt Team:	---	RHEL 7.3 requirements from Atomic Host:
Cloudforms Team:	---	Target Upstream Version:
Embargoed:
Bug Depends On:
Bug Blocks:	2041930

Description Kevin Fenzi 2022-01-25 17:26:15 UTC

From todays failed rawhide compose: 

08:29:49,894 DDEBUG dnf: RPM transaction start.
08:30:27,647 INF dnf.rpm: /var/tmp/rpm-tmp.zeypl5: line 1: ln: command not found
warning: %post(hunspell-1.7.0-13.fc36.x86_64) scriptlet failed, exit status 127

08:30:27,649 ERR dnf.rpm: Error in POSTIN scriptlet in rpm package hunspell


So, I guess you need a 'Requires: coreutils' or to change that post to use lua. 

'Requires: coreutils' might cause a loop...

Comment 1 Miro Hrončok 2022-01-25 17:28:51 UTC

This was introduced in https://src.fedoraproject.org/rpms/hunspell/c/09babb9078789004d270e8b7369279cdad64fd73?branch=rawhide maybe?

Seems like the package either needs Requires(post): coreutils (that should not cause a loop, or at least I don't know why) or to write both the scriptlets in Lua, not just the %pretrans one.

Comment 2 Jens Petersen 2022-01-26 03:21:15 UTC

I just pushed a hunspell-1.7.0-14.fc36 build to koji rawhide
with the Requires(post) change. Hopefully that is enough to
unbreak the compose.

(I wasn't smart enough to work out how to do symlinks easily in LUA.;-)

Comment 3 Miro Hrončok 2022-01-26 08:11:35 UTC

https://rpm-software-management.github.io/rpm/manual/lua.html suggests it is posix.symlink('existing', 'link')

Comment 4 Jens Petersen 2022-01-26 15:25:04 UTC

Thanks, Miro! That was helpful

I opened https://src.fedoraproject.org/rpms/hunspell/pull-request/4

which also adds an existence check to %post than was missing.

Comment 5 Jens Petersen 2022-01-26 15:32:47 UTC

I am also curious if relative or absolute symlink makes any difference here.
Probably not much in this case?  To be conservative we went with absolute so far.