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

Bug 830558

Summary: app-misc/qlcplus-4.12.4 fails to compile: velleman_mock.cpp:40:26: error: 'int32_t' was not declared in this scope
Product: Gentoo Linux Reporter: Agostino Sarubbo <ago>
Component: Current packagesAssignee: jannis <kripton>
Status: RESOLVED FIXED    
Severity: normal CC: proxy-maint
Priority: Normal Keywords: PullRequest
Version: unspecified   
Hardware: All   
OS: Linux   
See Also: https://github.com/gentoo/gentoo/pull/26074
Whiteboard:
Package list:
Runtime testing required: ---
Bug Depends on:    
Bug Blocks: 430702, 713786    
Attachments: build.log.xz

Description Agostino Sarubbo gentoo-dev 2022-01-03 20:47:29 UTC
https://blogs.gentoo.org/ago/2020/07/04/gentoo-tinderbox/

Issue: app-misc/qlcplus-4.12.4 fails to compile.
Discovered on: amd64 (internal ref: tinderbox_musl)

NOTE:
This machine uses MUSL libc
Comment 1 Agostino Sarubbo gentoo-dev 2022-01-03 20:47:31 UTC
Created attachment 761227 [details]
build.log.xz

build log and emerge --info (compressed because it exceeds attachment limit, use 'xzless' to read it)
Comment 2 jannis 2022-01-03 21:58:01 UTC
The report seems legitimate, regarding it's using musl libc.

Is there an easy way to reproduce the thinderbox's setup? Any scripts I could run to get to the same state before the emerge of app-misc/qlcplus?

I had a Dockerfile in the past but it didn't use musl. I'll see if I can re-use it.
Comment 3 jannis 2022-01-11 21:44:26 UTC
I was able to reproduce and fix this. PR created upstream:
https://github.com/mcallegari/qlcplus/pull/1311

Either I wait for the next release or I add this as patch to the ebuild and create a PR for Gentoo as well, will see :)
Comment 4 Brahmajit Das 2022-07-11 10:53:10 UTC
(In reply to jannis from comment #3)
> I was able to reproduce and fix this. PR created upstream:
> https://github.com/mcallegari/qlcplus/pull/1311
> 
> Either I wait for the next release or I add this as patch to the ebuild and
> create a PR for Gentoo as well, will see :)

Since this PR was merged, can we close this issue
Comment 5 Brahmajit Das 2022-07-11 10:54:19 UTC
(In reply to listout from comment #4)
> (In reply to jannis from comment #3)
> > I was able to reproduce and fix this. PR created upstream:
> > https://github.com/mcallegari/qlcplus/pull/1311
> > 
> > Either I wait for the next release or I add this as patch to the ebuild and
> > create a PR for Gentoo as well, will see :)
> 
> Since this PR was merged, can we close this issue

Okay, seems like the version bump is still pending
Comment 6 jannis 2022-07-11 12:04:05 UTC
(In reply to listout from comment #5)
> > Since this PR was merged, can we close this issue
> 
> Okay, seems like the version bump is still pending

Exactly, the PR unfortunately has not yet been merged. Sometimes it takes quite long for simple PRs to be reviewed :(

If a bug and a PR are linked (like in this case), the bug will automatically be closed when the PR has been merged.
Comment 7 Larry the Git Cow gentoo-dev 2022-08-04 06:37:24 UTC
The bug has been closed via the following commit(s):

https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=7709f4ea5be292d9d56a1fceae0727cd80d0e1cd

commit 7709f4ea5be292d9d56a1fceae0727cd80d0e1cd
Author:     Jannis Achstetter <kripton@kripserver.net>
AuthorDate: 2022-07-27 19:30:59 +0000
Commit:     Joonas Niilola <juippis@gentoo.org>
CommitDate: 2022-08-04 06:37:20 +0000

    app-misc/qlcplus: Bump to v4.12.5 using EAPI 8
    
    Package-Manager: Portage-3.0.34, Repoman-3.0.3
    Closes: https://bugs.gentoo.org/854765
    Closes: https://bugs.gentoo.org/830558
    Signed-off-by: Jannis Achstetter <kripton@kripserver.net>
    Signed-off-by: Joonas Niilola <juippis@gentoo.org>

 app-misc/qlcplus/Manifest              |  1 +
 app-misc/qlcplus/qlcplus-4.12.5.ebuild | 82 ++++++++++++++++++++++++++++++++++
 2 files changed, 83 insertions(+)