767379 – app-admin/needrestart: arm/arm64 keywording

Bug 767379 - app-admin/needrestart: arm/arm64 keywording

Summary: app-admin/needrestart: arm/arm64 keywording

Status:	RESOLVED FIXED

Alias:	None

Product:	Gentoo Linux
Classification:	Unclassified
Component:	Keywording (show other bugs)
Hardware:	All Linux

Importance:	Normal normal (vote)
Assignee:	Craig Andrews

URL:
Whiteboard:
Keywords:	CC-ARCHES

Depends on:
Blocks:

Reported:	2021-01-27 01:16 UTC by Sam James
Modified:	2021-03-30 17:37 UTC (History)
CC List:	1 user (show)

See Also:
Package list:	app-admin/needrestart ~arm ~arm64 dev-perl/Module-Find ~arm ~arm64 dev-perl/Module-ScanDeps ~arm ~arm64 dev-perl/Proc-ProcessTable ~arm ~arm64 dev-perl/Sort-Naturally ~arm ~arm64 sys-apps/init-system-helpers ~arm ~arm64
Runtime testing required:	---

Flags:	nattka: sanity-check+

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.

Description Sam James archtester

2021-01-27 01:16:18 UTC

Thanks!

Comment 1 NATTkA bot gentoo-dev

2021-01-27 01:20:57 UTC Comment hidden (obsolete)

Sanity check failed:

> app-admin/needrestart-3.5
>   depend ~arm stable profile default/linux/arm/17.0 (1 total)
>     sys-apps/init-system-helpers
>     sys-apps/iucode_tool
>   depend ~arm dev profile default/linux/arm/17.0/armv4 (35 total)
>     sys-apps/init-system-helpers
>     sys-apps/iucode_tool
>   rdepend ~arm stable profile default/linux/arm/17.0 (1 total)
>     sys-apps/init-system-helpers
>     sys-apps/iucode_tool
>   rdepend ~arm dev profile default/linux/arm/17.0/armv4 (35 total)
>     sys-apps/init-system-helpers
>     sys-apps/iucode_tool
>   depend ~arm64 stable profile default/linux/arm64/17.0 (9 total)
>     sys-apps/iucode_tool
>   rdepend ~arm64 stable profile default/linux/arm64/17.0 (9 total)
>     sys-apps/iucode_tool

Comment 2 Sam James archtester

2021-01-27 01:25:55 UTC

Hmm. sys-apps/iucode_tool doesn't make sense on non amd64/x86.

Comment 3 NATTkA bot gentoo-dev

2021-01-27 01:28:58 UTC Comment hidden (obsolete)

Sanity check failed:

> app-admin/needrestart-3.5
>   depend ~arm stable profile default/linux/arm/17.0 (10 total)
>     sys-apps/iucode_tool
>   depend ~arm dev profile default/linux/arm/17.0/armv4 (35 total)
>     sys-apps/iucode_tool
>   rdepend ~arm stable profile default/linux/arm/17.0 (10 total)
>     sys-apps/iucode_tool
>   rdepend ~arm dev profile default/linux/arm/17.0/armv4 (35 total)
>     sys-apps/iucode_tool

Comment 4 Sam James archtester

2021-02-01 23:24:47 UTC

(In reply to Sam James from comment #2)
> Hmm. sys-apps/iucode_tool doesn't make sense on non amd64/x86.

Craig, do you know if we could make this optional? It’s not compiled in so it might be as simple as either making a USE and forcing it on most arches or (probably better in this specific case given it won’t ever change...) !arm ( !arm64 ( ... ) )?

Comment 5 Sam James archtester

2021-03-15 00:42:43 UTC

(In reply to Sam James from comment #4)
> (In reply to Sam James from comment #2)
> > Hmm. sys-apps/iucode_tool doesn't make sense on non amd64/x86.
> 
> Craig, do you know if we could make this optional? It’s not compiled in so
> it might be as simple as either making a USE and forcing it on most arches
> or (probably better in this specific case given it won’t ever change...)
> !arm ( !arm64 ( ... ) )?

ping

Comment 6 Larry the Git Cow gentoo-dev

2021-03-17 02:11:49 UTC

The bug has been referenced in the following commit(s):

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

commit c19cf4651ac0718d92dff6fc5b904089395dfc28
Author:     Craig Andrews <candrews@gentoo.org>
AuthorDate: 2021-03-17 02:09:02 +0000
Commit:     Craig Andrews <candrews@gentoo.org>
CommitDate: 2021-03-17 02:11:36 +0000

    app-admin/needrestart: only x86/amd64 depend on sys-apps/iucode_tool
    
    Bug: https://bugs.gentoo.org/767379
    Package-Manager: Portage-3.0.17, Repoman-3.0.2
    Signed-off-by: Craig Andrews <candrews@gentoo.org>

 app-admin/needrestart/needrestart-3.5.ebuild  | 3 ++-
 app-admin/needrestart/needrestart-9999.ebuild | 3 ++-
 2 files changed, 4 insertions(+), 2 deletions(-)

Comment 7 Sam James archtester

2021-03-17 02:19:06 UTC

Thanks a bunch mate! \o/

Comment 8 Sam James archtester

2021-03-28 07:11:16 UTC

arm done

Comment 9 Sam James archtester

2021-03-30 17:37:37 UTC

arm64 done

all arches done