Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 881771 - sys-apps/tas: drop x86 keyword
Summary: sys-apps/tas: drop x86 keyword
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: All Linux
: Normal normal
Assignee: Conrad Kostecki
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2022-11-18 08:12 UTC by Joonas Niilola
Modified: 2022-11-20 06:57 UTC (History)
0 users

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


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Joonas Niilola gentoo-dev 2022-11-18 08:12:59 UTC
sys-apps/tas x86 depends on a binary-only sys-block/storcli which has dropped x86 support years ago. This is an attempt to see if anyone still uses tas. If yes, we'll need to revert commit 05186ae00cc0945a8f44358854db6f1b14cd68d8 and restore storcli-7.1316. Otherwise, x86 de-keywording will happen in some time (+30 days from now).
Comment 1 Larry the Git Cow gentoo-dev 2022-11-18 08:17:58 UTC
The bug has been referenced in the following commit(s):

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

commit c5a292a42f45830e9b94d2c3fa350e3e870a7c92
Author:     Joonas Niilola <juippis@gentoo.org>
AuthorDate: 2022-11-18 08:13:28 +0000
Commit:     Joonas Niilola <juippis@gentoo.org>
CommitDate: 2022-11-18 08:17:54 +0000

    profiles/arch/x86: mask sys-apps/tas
    
    Bug: https://bugs.gentoo.org/881771
    Signed-off-by: Joonas Niilola <juippis@gentoo.org>

 profiles/arch/x86/package.mask | 4 ++++
 1 file changed, 4 insertions(+)
Comment 2 Conrad Kostecki gentoo-dev 2022-11-19 21:44:03 UTC
It would have been really nice, if I would have been asked prior removing package and masking it..

Anyway, it looks like that storcli seems finally optional. This wasn't the case when I added it. Will move it to optfeature.
Comment 3 Larry the Git Cow gentoo-dev 2022-11-19 21:52:26 UTC
The bug has been closed via the following commit(s):

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

commit 46bf9e80840a1621279ebc6ef5915102f5505515
Author:     Conrad Kostecki <conikost@gentoo.org>
AuthorDate: 2022-11-19 21:50:09 +0000
Commit:     Conrad Kostecki <conikost@gentoo.org>
CommitDate: 2022-11-19 21:50:09 +0000

    sys-apps/tas: drop storcli from RDEPEND
    
    Closes: https://bugs.gentoo.org/881771
    Signed-off-by: Conrad Kostecki <conikost@gentoo.org>

 sys-apps/tas/{tas-1.7.0.220503.ebuild => tas-1.7.0.220503-r1.ebuild} | 1 -
 1 file changed, 1 deletion(-)

Additionally, it has been referenced in the following commit(s):

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

commit 9ec1564b2541226f2d4ed45f87506a1151da0977
Author:     Conrad Kostecki <conikost@gentoo.org>
AuthorDate: 2022-11-19 21:51:01 +0000
Commit:     Conrad Kostecki <conikost@gentoo.org>
CommitDate: 2022-11-19 21:52:11 +0000

    Revert "profiles/arch/x86: mask sys-apps/tas"
    
    This reverts commit c5a292a42f45830e9b94d2c3fa350e3e870a7c92.
    
    Bug: https://bugs.gentoo.org/881771
    Signed-off-by: Conrad Kostecki <conikost@gentoo.org>

 profiles/arch/x86/package.mask | 4 ----
 1 file changed, 4 deletions(-)
Comment 4 Joonas Niilola gentoo-dev 2022-11-20 06:57:32 UTC
(In reply to Conrad Kostecki from comment #2)
> It would have been really nice, if I would have been asked prior removing
> package and masking it..
> 
> Anyway, it looks like that storcli seems finally optional. This wasn't the
> case when I added it. Will move it to optfeature.

Then remove your devaway ;) but yes I admit this dependency came a bit of a surprise too, had to work more than usual to get newer storcli merged so just wanted to "finish it". 
https://github.com/gentoo/gentoo/pull/28002