Bug 640274 – x11-wm/i3: add ~arm64 keyword

• Status: RESOLVED FIXED
• Product: Gentoo Linux

[Description Conrad Kostecki 2017-12-08 16:19:23 UTC]

Please add ~arm64 keyword for x11-wm/i3.
Compiles here fine on my Raspberry Pi 3.

[Comment 1 Nelo-T. Wallus (ntnn) 2017-12-17 14:04:44 UTC]

Could you attach the build log? Thanks.

[Comment 2 Conrad Kostecki 2017-12-17 15:08:22 UTC]

Created attachment 510484 [details]
build.log

(In reply to Nelo-T. Wallus (ntnn) from comment #1)
> Could you attach the build log? Thanks.

Sure, here we go :)

[Comment 3 Markus Meier 2017-12-24 12:02:32 UTC]

~arm64 added, closing.

[Comment 4 Mart Raudsepp 2018-02-11 10:03:23 UTC]

  dependency.badindev           2
   x11-wm/i3/i3-4.14.1.ebuild: DEPEND: ~arm64(default/linux/arm64/17.0)
[     '>=dev-perl/X11-XCB-0.120.0',
      'dev-perl/Inline-C',
      'dev-perl/IPC-Run',
      'dev-perl/local-lib']
   x11-wm/i3/i3-4.14.1.ebuild: DEPEND: ~arm64(default/linux/arm64/17.0/desktop/systemd)
[     '>=dev-perl/X11-XCB-0.120.0',
      'dev-perl/Inline-C',
      'dev-perl/IPC-Run',
      'dev-perl/local-lib']

[Comment 5 Mart Raudsepp 2018-02-11 10:04:05 UTC]

Make sure to run test suites...

[Comment 6 Larry the Git Cow 2018-02-14 19:38:41 UTC]

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

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

commit 4df6d49f6f74e5cc7a52d10341689df4f9c78489
Author:     Nelo-T. Wallus <[e-mail]>
AuthorDate: 2018-02-14 17:43:42 +0000
Commit:     Michał Górny <[e-mail]>
CommitDate: 2018-02-14 19:38:03 +0000

    x11-wm/i3: Revert "add ~arm64, bug #640274"
    
    Dependencies are not keyworded, see bug for details.
    
    Bug: https://bugs.gentoo.org/640274
    Closes: https://github.com/gentoo/gentoo/pull/7187
    Reverts: 5bde02429fa44ae1d62e116b7ae008ec6b3babd0

 x11-wm/i3/i3-4.14.1.ebuild | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)}

[Comment 7 Nelo-T. Wallus (ntnn) 2018-02-20 18:03:57 UTC]

amd64, please test the dependencies.
Conrad, if you have the time testing the dependencies and attaching their build logs would be great. Thanks.

[Comment 8 Conrad Kostecki 2018-02-20 22:23:38 UTC]

Hi!
Maybe I am totally wrong, but when I do want to emerge dev-perl/X11-XCB, dev-perl/Inline-C, dev-perl/IPC-Run, dev-perl/local-lib, there are a lot of more dependencies, which need to be keyworded, at least portage complains.

Can be this possible?

dev-perl/local-lib
dev-perl/IPC-Run
dev-perl/Inline-C
dev-perl/X11-XCB
dev-perl/Data-Dump
dev-perl/Mouse
dev-perl/Module-Build-XSUtil
dev-perl/Devel-CheckCompiler
dev-perl/Module-Build-Tiny
dev-perl/ExtUtils-Config
dev-perl/ExtUtils-Helpers
dev-perl/ExtUtils-InstallPaths
dev-perl/MouseX-NativeTraits
dev-perl/Any-Moose
dev-perl/Moose
dev-perl/Class-Load
dev-perl/Class-Load-XS
dev-perl/Devel-GlobalDestruction
dev-perl/Devel-OverloadInfo
dev-perl/Module-Runtime-Conflicts
dev-perl/XML-Descent
dev-perl/XML-TokeParser
dev-perl/XS-Object-Magic
dev-perl/Pegex
dev-perl/File-ShareDir-Install

Conrad

[Comment 9 Nelo-T. Wallus (ntnn) 2018-02-20 22:34:26 UTC]

Yes, that's entirely possible. I'll compile the complete list of required dependencies and update the package list tommorrow.

[Comment 10 Conrad Kostecki 2018-02-20 23:18:36 UTC]

Created attachment 520352 [details]
build.log.tar

Here are the build.log files of all packages, I build.

[Comment 11 Nelo-T. Wallus (ntnn) 2018-02-21 21:22:09 UTC]

Thanks Conrad! There are some more packages though, if you have the time please build them and attach the logs:

=dev-perl/Cwd-Guard-0.50.0
=dev-perl/File-Copy-Recursive-0.380.0-r1
=dev-perl/MooseX-Types-0.500.0
=dev-perl/Test-CleanNamespaces-0.180.0
=dev-perl/Test-Differences-0.640.0
=dev-perl/YAML-LibYAML-0.650.0
=dev-perl/File-Copy-Link-0.140.0
=dev-perl/Sub-Exporter-ForMethods-0.100.52
=dev-perl/File-Find-Rule-0.340.0
=dev-perl/File-Find-Rule-Perl-1.150.0
=dev-perl/Number-Compare-0.30.0-r1
=dev-perl/Text-Glob-0.90.0-r1
=dev-perl/Test-Differences-0.640.0
=dev-perl/Text-Diff-1.440.0

[Comment 12 Conrad Kostecki 2018-02-21 23:50:07 UTC]

Created attachment 520518 [details]
build.log.2.tar

(In reply to Nelo-T. Wallus (ntnn) from comment #11)
> Thanks Conrad! There are some more packages though, if you have the time
> please build them and attach the logs:
> 
> =dev-perl/Cwd-Guard-0.50.0
> =dev-perl/File-Copy-Recursive-0.380.0-r1
> =dev-perl/MooseX-Types-0.500.0
> =dev-perl/Test-CleanNamespaces-0.180.0
> =dev-perl/Test-Differences-0.640.0
> =dev-perl/YAML-LibYAML-0.650.0
> =dev-perl/File-Copy-Link-0.140.0
> =dev-perl/Sub-Exporter-ForMethods-0.100.52
> =dev-perl/File-Find-Rule-0.340.0
> =dev-perl/File-Find-Rule-Perl-1.150.0
> =dev-perl/Number-Compare-0.30.0-r1
> =dev-perl/Text-Glob-0.90.0-r1
> =dev-perl/Test-Differences-0.640.0
> =dev-perl/Text-Diff-1.440.0

After setting ~arm64 keyword, all packages build fine.

[Comment 13 Markus Meier 2018-03-15 21:15:02 UTC]

(In reply to Mart Raudsepp from comment #5)
> Make sure to run test suites...

fun stuff - test is RESTRICTED... I guess that's why I initially didn't notice the dependencies (no clue how repoman let me commit this).

[Comment 14 Stabilization helper bot 2018-04-27 23:00:03 UTC]

An automated check of this bug failed - the following atom is unknown:

dev-perl/X11-XCB-0.160.0

Please verify the atom list.

[Comment 15 Stabilization helper bot 2018-07-10 19:01:51 UTC]

An automated check of this bug succeeded - the previous repoman errors are now resolved.

[Comment 16 Johannes Huber (RETIRED) 2018-07-17 06:03:26 UTC]

FYI arm was dropped too...

RepoMan scours the neighborhood...
  KEYWORDS.dropped              1
   x11-wm/i3/i3-4.15.ebuild: arm

[Comment 17 Conrad Kostecki 2018-07-17 13:24:27 UTC]

(In reply to Johannes Huber from comment #16)
> FYI arm was dropped too...
> 
> RepoMan scours the neighborhood...
>   KEYWORDS.dropped              1
>    x11-wm/i3/i3-4.15.ebuild: arm

Hm, I see. This got removed by 4.14.1. Could we reintroduce this again?

[Comment 18 Conrad Kostecki 2019-03-02 17:02:17 UTC]

Closing, as current version do have enable arm64 keyword.