Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 762601 - media-gfx/valentina-0.6.1-r1 calls commands that do not exist - hg: command not found
Summary: media-gfx/valentina-0.6.1-r1 calls commands that do not exist - hg: command n...
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: All Linux
: Normal normal (vote)
Assignee: Miroslav Šulc
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2020-12-30 08:56 UTC by Agostino Sarubbo
Modified: 2021-01-28 12:45 UTC (History)
1 user (show)

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


Attachments
build.log.xz (build.log.xz,30.01 KB, application/x-xz)
2020-12-30 08:56 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 2020-12-30 08:56:44 UTC
https://blogs.gentoo.org/ago/2020/07/04/gentoo-tinderbox/

Issue: media-gfx/valentina-0.6.1-r1 calls commands that do not exist.
Discovered on: amd64 (internal ref: ci)
Comment 1 Agostino Sarubbo gentoo-dev 2020-12-30 08:56:47 UTC
Created attachment 680260 [details]
build.log.xz

build log and emerge --info (compressed because it exceeds attachment limit, use 'xzless' to read it)
Comment 2 Miroslav Šulc gentoo-dev 2020-12-30 09:40:58 UTC
Roman, could you please have a look at this? it's the code in common.pri that attempts to fill the DVCS_HESH variable. users might not have hg nor git installed on their system and though this issue is not fatal, it could be avoided by checking whether the command is available. what do you think?
Comment 3 Roman 2020-12-30 15:16:33 UTC
In case hg or git is not installed script just sets a hash value to "unknown". So, I don't see a reason to add something else.
Comment 4 Miroslav Šulc gentoo-dev 2020-12-30 15:43:36 UTC
if i get it right, there is no check whether the commands are available or not, they are just called and if they fail, it works as you wrote. nontheless this report is about the error output when those commands are called but not installed. this is from the compilation output:

sh: line 1: hg: command not found

i can sure patch it out for the versioned releases as there is no hg/git repo anyway, and also patch out hg for the live ebuild as it's git based. i just thought it might be better to add a check into common.pri whether the command is available or not to avoid the error message "command not found".
Comment 5 Roman 2020-12-30 20:46:49 UTC
Do you have an idea of how to add such a check? Because my approach was to call, check a result, react to it. This is why you see this error message that just means you don't have hg/git, nothing more. Not critical and must be just ignored.
Comment 6 Miroslav Šulc gentoo-dev 2020-12-30 20:51:26 UTC
not at all. but i can simply patch it out to avoid the error (warning) :-)
Comment 7 Roman 2020-12-30 20:53:46 UTC
Okay. I will add a config option to help people to suppress if they wish not to add a hash value.
Comment 8 Miroslav Šulc gentoo-dev 2020-12-30 20:55:09 UTC
thanks! that'd be great :-) could you also switch the order to git -> hg instead of hg -> git?
Comment 9 Miroslav Šulc gentoo-dev 2020-12-30 20:56:18 UTC
i mean to first check for git output and then for hg. in that case i could still use the check for the live ebuild.
Comment 10 Roman 2020-12-30 20:59:02 UTC
It will take some time. I am very busy these days.
Comment 11 Miroslav Šulc gentoo-dev 2020-12-30 21:01:19 UTC
no worries at all. this is a trivial issue from the point of view of users, it's rather qa issue. it does not affect installing and using valentina at all.
Comment 12 Roman 2021-01-23 11:18:53 UTC
We no longer have calls to HG in develop and 0.6.x branches. New releases will fix this issue. But you can also create a patch yourself based on the current implementation.
Comment 13 Larry the Git Cow gentoo-dev 2021-01-28 12:45:28 UTC
The bug has been closed via the following commit(s):

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

commit bfcfbfe0dacfc3ee5a6601a4f85f206309fc09de
Author:     Miroslav Šulc <fordfrog@gentoo.org>
AuthorDate: 2021-01-28 12:45:02 +0000
Commit:     Miroslav Šulc <fordfrog@gentoo.org>
CommitDate: 2021-01-28 12:45:02 +0000

    media-gfx/valentina: patched out vcs calls in 0.6.1-r1
    
    Closes: https://bugs.gentoo.org/762601
    Package-Manager: Portage-3.0.14, Repoman-3.0.2
    Signed-off-by: Miroslav Šulc <fordfrog@gentoo.org>

 .../valentina-0.6.1-disable-vcs-version.patch      | 75 ++++++++++++++++++++++
 media-gfx/valentina/valentina-0.6.1-r1.ebuild      |  3 +-
 2 files changed, 77 insertions(+), 1 deletion(-)