Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 937496 - sys-power/apcupsd-3.14.14-r3 calls commands that do not exist: col
Summary: sys-power/apcupsd-3.14.14-r3 calls commands that do not exist: col
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: All Linux
: Normal normal
Assignee: John Einar Reitan
URL:
Whiteboard:
Keywords: PullRequest
: 938549 (view as bug list)
Depends on:
Blocks:
 
Reported: 2024-08-07 07:35 UTC by Agostino Sarubbo
Modified: 2025-01-19 00:20 UTC (History)
4 users (show)

See Also:
Package list:
Runtime testing required: ---


Attachments
build.log (build.log,115.42 KB, text/plain)
2024-08-07 07:35 UTC, Agostino Sarubbo
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Agostino Sarubbo gentoo-dev 2024-08-07 07:35:53 UTC
https://blogs.gentoo.org/ago/2020/07/04/gentoo-tinderbox/

Issue: sys-power/apcupsd-3.14.14-r3 calls commands that do not exist.
Discovered on: amd64 (internal ref: tinderbox_musl)
System: MUSL-SYSTEM (https://wiki.gentoo.org/wiki/Project:Tinderbox/Common_Issues_Helper#MUSL)

Info about the issue:
https://wiki.gentoo.org/wiki/Project:Tinderbox/Common_Issues_Helper#QA0018
Comment 1 Agostino Sarubbo gentoo-dev 2024-08-07 07:35:54 UTC
Created attachment 899345 [details]
build.log

build log and emerge --info
Comment 2 John Einar Reitan 2024-08-07 08:36:00 UTC
I see it fails on col, which is part of sys-apps/util-linux, but that package is listed as installed. So is something messed up?
Comment 3 Agostino Sarubbo gentoo-dev 2024-08-07 08:57:00 UTC
qlist -e sys-apps/util-linux for version 2.40.2 does not list "col" while stable version (2.39*) does.

Dunno if it was removed or it gets installed with an IUSE
Comment 4 Toralf Förster gentoo-dev 2024-08-27 11:30:58 UTC
*** Bug 938549 has been marked as a duplicate of this bug. ***
Comment 5 John Einar Reitan 2024-08-27 14:51:25 UTC
I see 2.40 stopped building col on non-glibc:
://github.com/util-linux/util-linux/blob/master/Documentation/releases/v2.40-ReleaseNotes
Comment 6 John Einar Reitan 2024-08-27 14:52:38 UTC
Sic, mangled url, correct URL is:

https://github.com/util-linux/util-linux/blob/master/Documentation/releases/v2.40-ReleaseNotes
Comment 7 John Einar Reitan 2024-09-18 14:54:05 UTC
I guess col could affect multiple packages, should we be patching all of them, add some kind of substitute version to install on musl or what?
Comment 8 Holger Hoffstätte 2024-09-18 20:31:52 UTC
I was wondering why it would be calling 'col' in the first place, and as expected it can just be ignored/deleted.

From autoconf/targets.mak:

  # Format a manpage into plain text
  define MANIFY
      @$(ECHO) "  MAN  " $(1) -\> $(2)
      $(V)man ./$(1) | col -b > $(2)
  endef

This MANIFY is called several times from doc/Makefile, for the all-targets target which converts the prebuilt man pages into text. These are never installed by the ebuild.

The easiest fix here is to simply remove the doc target from the top-level SUBDIRS in Makefile:

  sed -i '^SUBDIRS/ s/doc//g" Makefile || die

Everything else is installed by the ebuild.
Comment 9 John Einar Reitan 2024-09-19 07:20:12 UTC
Ok, I'll try that and make a pull request if working.
Comment 10 Larry the Git Cow gentoo-dev 2025-01-19 00:20:21 UTC
The bug has been closed via the following commit(s):

https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=31c1a238c60eaa092cb674008c77d0644dde09e5

commit 31c1a238c60eaa092cb674008c77d0644dde09e5
Author:     John Einar Reitan <john.einar@gmail.com>
AuthorDate: 2024-09-19 08:18:35 +0000
Commit:     Conrad Kostecki <conikost@gentoo.org>
CommitDate: 2025-01-19 00:20:18 +0000

    sys-power/apcupsd: Remove dependency on col command
    
    Closes: https://bugs.gentoo.org/937496
    
    Signed-off-by: John Einar Reitan <john.einar@gmail.com>
    Closes: https://github.com/gentoo/gentoo/pull/38681
    Signed-off-by: Conrad Kostecki <conikost@gentoo.org>

 sys-power/apcupsd/apcupsd-3.14.14-r3.ebuild | 7 +++++++
 1 file changed, 7 insertions(+)