Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 832775 - dev-lang/spidermonkey-68.12.0 depends on <dev-libs/icu-70, blocking cleanup
Summary: dev-lang/spidermonkey-68.12.0 depends on <dev-libs/icu-70, blocking cleanup
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: All Linux
: Normal normal (vote)
Assignee: Mozilla Gentoo Team
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2022-02-05 23:20 UTC by Andreas Sturmlechner
Modified: 2022-02-07 13:13 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 Andreas Sturmlechner gentoo-dev 2022-02-05 23:20:24 UTC
Please try to change that.
Comment 1 Larry the Git Cow gentoo-dev 2022-02-05 23:24:48 UTC
The bug has been referenced in the following commit(s):

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

commit 6042abbd224797421d5eedef472f9f157632c724
Author:     Andreas Sturmlechner <asturm@gentoo.org>
AuthorDate: 2022-02-05 23:21:37 +0000
Commit:     Andreas Sturmlechner <asturm@gentoo.org>
CommitDate: 2022-02-05 23:24:25 +0000

    dev-libs/icu-layoutex: Revert "Drop 69.1"
    
    This reverts commit af0988e1ef54e824d29ee58dc233ebecb88518c9.
    
    Bug: https://bugs.gentoo.org/832775
    Signed-off-by: Andreas Sturmlechner <asturm@gentoo.org>

 dev-libs/icu-layoutex/Manifest                 |   1 +
 dev-libs/icu-layoutex/icu-layoutex-69.1.ebuild | 109 +++++++++++++++++++++++++
 2 files changed, 110 insertions(+)

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

commit 28fee867fc829a7d20395c8b3830615d948e9c3e
Author:     Andreas Sturmlechner <asturm@gentoo.org>
AuthorDate: 2022-02-05 23:18:24 +0000
Commit:     Andreas Sturmlechner <asturm@gentoo.org>
CommitDate: 2022-02-05 23:24:15 +0000

    dev-libs/icu: Revert "Drop 69.1-r1"
    
    This reverts commit f9e554b249cde1ae33d6e09a9fb0b22bcc9aa9c0.
    
    Bug: https://bugs.gentoo.org/832775
    Signed-off-by: Andreas Sturmlechner <asturm@gentoo.org>

 dev-libs/icu/Manifest           |   1 +
 dev-libs/icu/icu-69.1-r1.ebuild | 150 ++++++++++++++++++++++++++++++++++++++++
 2 files changed, 151 insertions(+)
Comment 2 Joonas Niilola gentoo-dev 2022-02-06 07:09:38 UTC
So AFAIR spidermonkey:68 didn't build with icu-70 (probably failed already at configure state). But succeeded with 69. 

I can try to find a patch or perhaps write one, although it'd be unsupported and require dropping keywords from stable. I think the bigger issue here is we can't clean <spidermonkey:78 because there are still packages depending on those, and >=:78 requires rust.
Comment 3 Joonas Niilola gentoo-dev 2022-02-06 07:23:34 UTC
Or we just use the bundled icu for 68...
Comment 4 Andreas Sturmlechner gentoo-dev 2022-02-07 12:04:17 UTC
(In reply to Joonas Niilola from comment #2)
> So AFAIR spidermonkey:68 didn't build with icu-70 (probably failed already
> at configure state). But succeeded with 69. 
And no one filed a bug about that after more than 2 months in ~arch? Come on.

I've just started --nodeps build and configure it did fine.
Build it also did fine.
Comment 5 Joonas Niilola gentoo-dev 2022-02-07 13:01:57 UTC
(In reply to Andreas Sturmlechner from comment #4)
> And no one filed a bug about that after more than 2 months in ~arch? Come on.
> 

Thought no ~ package depended on :68 anymore, but I found one! (polkit)

> I've just started --nodeps build and configure it did fine.
> Build it also did fine.

Thanks, I also tried again and it's amazing what you'll figure when you focus on the task at hand, unlike having to fix unrelated errors to what you're trying to do! You can now have your cleanup.
Comment 6 Larry the Git Cow gentoo-dev 2022-02-07 13:03:18 UTC
The bug has been closed via the following commit(s):

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

commit 8184a35e5575e7a5283648601a36036f30b3ec19
Author:     Joonas Niilola <juippis@gentoo.org>
AuthorDate: 2022-02-07 12:58:33 +0000
Commit:     Joonas Niilola <juippis@gentoo.org>
CommitDate: 2022-02-07 13:03:12 +0000

    dev-lang/spidermonkey: unrestict icu in 68, add missing BDEPEND
    
    Closes: https://bugs.gentoo.org/832775
    Signed-off-by: Joonas Niilola <juippis@gentoo.org>

 .../{spidermonkey-68.12.0.ebuild => spidermonkey-68.12.0-r1.ebuild}    | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)
Comment 7 Andreas Sturmlechner gentoo-dev 2022-02-07 13:05:03 UTC
(In reply to Joonas Niilola from comment #5)
> Thanks, I also tried again and it's amazing what you'll figure when you
> focus on the task at hand, unlike having to fix unrelated errors to what
> you're trying to do! You can now have your cleanup.
In the future please only add upper bounds version limits as a very last resort.
Comment 8 Andreas Sturmlechner gentoo-dev 2022-02-07 13:06:30 UTC
No more subslot operator?
Comment 9 Larry the Git Cow gentoo-dev 2022-02-07 13:10:20 UTC
The bug has been referenced in the following commit(s):

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

commit 771a0e44e123805ee9ae792ddf284e83eec5b170
Author:     Joonas Niilola <juippis@gentoo.org>
AuthorDate: 2022-02-07 13:09:44 +0000
Commit:     Joonas Niilola <juippis@gentoo.org>
CommitDate: 2022-02-07 13:09:44 +0000

    dev-lang/spidermonkey: add subslot binder to icu on 68
    
    Bug: https://bugs.gentoo.org/832775
    Signed-off-by: Joonas Niilola <juippis@gentoo.org>

 dev-lang/spidermonkey/spidermonkey-68.12.0-r1.ebuild | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
Comment 10 Joonas Niilola gentoo-dev 2022-02-07 13:12:17 UTC
Allow me to rephrase the following: 
> 
> Thanks, I also tried again and it's amazing what you'll figure when you
> focus on the task at hand, unlike having to fix unrelated errors to what
> you're trying to do! You can now have your cleanup.

Into: F*** mondays.

:)

Thanks for catching that.

(
  >>> Original instance of package unmerged safely.
   * VDB: detected possibly incorrect RDEPEND (dev-lang/spidermonkey-68.12.0-r1)
   * dev-libs/icu | dev-libs/icu:=
)
Comment 11 Larry the Git Cow gentoo-dev 2022-02-07 13:13:26 UTC
The bug has been referenced in the following commit(s):

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

commit b1a6ce380ac411207b9a44c5f2f2dac5c10eae7a
Author:     Andreas Sturmlechner <asturm@gentoo.org>
AuthorDate: 2022-02-05 20:23:41 +0000
Commit:     Andreas Sturmlechner <asturm@gentoo.org>
CommitDate: 2022-02-07 13:13:14 +0000

    dev-libs/icu: Drop 69.1-r1 (again)
    
    Bug: https://bugs.gentoo.org/832775
    Package-Manager: Portage-3.0.30, Repoman-3.0.3
    Signed-off-by: Andreas Sturmlechner <asturm@gentoo.org>
    (cherry picked from commit f9e554b249cde1ae33d6e09a9fb0b22bcc9aa9c0)

 dev-libs/icu/Manifest           |   1 -
 dev-libs/icu/icu-69.1-r1.ebuild | 150 ----------------------------------------
 2 files changed, 151 deletions(-)

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

commit 95da1fcee2fec7e61d7e112ade4ab9e009b5137d
Author:     Andreas Sturmlechner <asturm@gentoo.org>
AuthorDate: 2022-02-05 20:22:40 +0000
Commit:     Andreas Sturmlechner <asturm@gentoo.org>
CommitDate: 2022-02-07 13:13:14 +0000

    dev-libs/icu-layoutex: Drop 69.1 (again)
    
    Bug: https://bugs.gentoo.org/832775
    Package-Manager: Portage-3.0.30, Repoman-3.0.3
    Signed-off-by: Andreas Sturmlechner <asturm@gentoo.org>
    (cherry picked from commit af0988e1ef54e824d29ee58dc233ebecb88518c9)

 dev-libs/icu-layoutex/Manifest                 |   1 -
 dev-libs/icu-layoutex/icu-layoutex-69.1.ebuild | 109 -------------------------
 2 files changed, 110 deletions(-)