Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!

Bug 933185

Summary: [guru] net-dialup/syncterm-1.1 calls strip directly
Product: GURU Reporter: Agostino Sarubbo <ago>
Component: Package issuesAssignee: Seth M. Price <sprice623>
Status: RESOLVED TEST-REQUEST    
Severity: normal CC: guru-bugs, sprice623
Priority: Normal    
Version: unspecified   
Hardware: All   
OS: Linux   
Whiteboard:
Package list:
Runtime testing required: ---
Bug Depends on:    
Bug Blocks: 785607    
Attachments: build.log

Description Agostino Sarubbo gentoo-dev 2024-05-30 10:22:41 UTC
https://blogs.gentoo.org/ago/2020/07/04/gentoo-tinderbox/

Issue: net-dialup/syncterm-1.1 calls strip directly.
Discovered on: amd64 (internal ref: guru_ci)

Info about the issue:
https://wiki.gentoo.org/wiki/Project:Tinderbox/Common_Issues_Helper#QA0014
Comment 1 Agostino Sarubbo gentoo-dev 2024-05-30 10:22:42 UTC
Created attachment 894649 [details]
build.log

build log and emerge --info
Comment 2 Agostino Sarubbo gentoo-dev 2024-05-30 10:22:44 UTC
Error(s) that match a know pattern:


./tools/buildlib.sh: line 34: ar: command not found
./tools/ccopts.sh: line 51: strip: command not found
Comment 3 Seth M. Price 2024-07-08 21:11:14 UTC
Just to clarify: I’m still working on this, on and off.  I’m trying to figure out how to reasonably patch this whole thing, and it’s quite a challenge for me.
Comment 4 Viorel Munteanu gentoo-dev 2024-11-25 17:24:22 UTC
Did you try the simple solutions?

inherit toolchain-funcs

tc-export AR

This may take care or ar.

As for strip, simply remove that line from ccopts.sh, portage calls strip at the end if needed.
Comment 5 Sam James archtester Gentoo Infrastructure gentoo-dev Security 2024-11-25 17:28:54 UTC
WONTFIX also really needs some rationale accompanying it. If you're stuck, ask for help.
Comment 6 Seth M. Price 2024-11-25 17:33:02 UTC
I'll look back into it with a fresh mind within the day.  I was afraid of being ridiculed for asking for help, but it seems that's not the case.  :)
Comment 7 Seth M. Price 2024-11-26 19:21:55 UTC
The problematic file is inside a .zip archive.  I think I can fix it by patching `${S}/3rdp/build/GNUMakefile', but `files/syncterm-1.1_use-sched-yield-instead-of-pthread-yield.patch' already does something to the lines that need changed.  Is it bad practice to write a patch that depends on another patch having been applied?
Comment 8 Larry the Git Cow gentoo-dev 2024-11-27 23:25:11 UTC
The bug has been referenced in the following commit(s):

https://gitweb.gentoo.org/repo/proj/guru.git/commit/?id=5b84c0f56ba3b8aa9010d5978890d07f086e6d8d

commit 5b84c0f56ba3b8aa9010d5978890d07f086e6d8d
Author:     Seth M. Price <sprice623@aol.com>
AuthorDate: 2024-11-27 22:32:09 +0000
Commit:     Seth M. Price <sprice623@aol.com>
CommitDate: 2024-11-27 22:34:40 +0000

    net-dialup/syncterm: fix various QA issues
    
    cryptlib’s build system is awful.  Hopefully this is the last
    of what I need to do to avoid QA issues on the automated testing
    box thingy; I’ll just wait and see.
    
    No revision bump since nothing changed for users who successfully
    built it before.
    
    Bug: https://bugs.gentoo.org/933185
    Bug: https://bugs.gentoo.org/933184
    Signed-off-by: Seth M. Price <sprice623@aol.com>

 .../syncterm-1.1_do-not-strip-3rdp-test.patch      | 40 ++++++++++
 .../syncterm-1.1_respect-toolchain-variables.patch | 89 ++++++++++++++++++++++
 net-dialup/syncterm/syncterm-1.1.ebuild            |  5 +-
 3 files changed, 133 insertions(+), 1 deletion(-)