Skip to content

build: fix indeterminacy of icu_locales value - #42865

Merged
nodejs-github-bot merged 1 commit into
nodejs:masterfrom
3ap:patch-1
May 6, 2022
Merged

build: fix indeterminacy of icu_locales value#42865
nodejs-github-bot merged 1 commit into
nodejs:masterfrom
3ap:patch-1

Conversation

@3ap

@3ap3ap commented Apr 25, 2022

Copy link
Copy Markdown
Contributor

icu_locales is generated by joining values from set data structure.
However, set doesn't guarantee an order, so the result of icu_locales is not determined.
For example, the result value could be 'en,root' or 'root,en'. This fix make it deterministic.

The main reason of this fix is to restore the reproducibility of the build because the value of icu_locales is embedded into node binary.

The minimal code below could show you the issue:

default='root,en'locs=set(default.split(','))
locs.add('root')
result=','.join(str(loc) forlocinlocs)
print(result)

Even after several runs of this script the problem occurs:

→ python test.py
en,root
→ python test.py
root,en
→ python test.py
en,root

@nodejs-github-botnodejs-github-bot added build Issues and PRs related to build files or the CI. needs-ci PRs that need a full CI run. labels Apr 25, 2022
@richardlaurichardlau added the request-ci Add this label to start a Jenkins CI on a PR. label Apr 25, 2022
@3ap
build: fix indeterminacy of icu_locales value
bd69f29
`icu_locales` is generated by joining values from `set` data structure.
However, `set` doesn't guarantee an order, so the result of
`icu_locales` is not determined. For example, the result value could be
'en,root' or 'root,en'. This fix makes it deterministic.
The main reason of this fix is to restore the reproducibility of the
build because the value of `icu_locales` is embedded into `node` binary.
@github-actionsgithub-actionsBot removed the request-ci Add this label to start a Jenkins CI on a PR. label Apr 25, 2022
@nodejs-github-bot

This comment was marked as outdated.

Sign in to view
@nodejs-github-bot

nodejs-github-bot commented Apr 25, 2022

Copy link
Copy Markdown
Collaborator

CI: https://ci.nodejs.org/job/node-test-pull-request/43687/

@github-actionsgithub-actionsBot mentioned this pull request Apr 26, 2022
Open
14 tasks
@nodejs-github-bot

nodejs-github-bot commented Apr 26, 2022

Copy link
Copy Markdown
Collaborator

CI: https://ci.nodejs.org/job/node-test-pull-request/43705/

This was referenced Apr 27, 2022
Open
Open
@3ap

3ap commented Apr 28, 2022•
edited
Loading

Copy link
Copy Markdown
ContributorAuthor

I'm not sure I understand what's wrong with CI and failed tests, could anybody explain it to me? Should or could I do something to fix it?

@github-actionsgithub-actionsBot mentioned this pull request Apr 29, 2022
Open
21 tasks
@nodejs-github-bot

nodejs-github-bot commented Apr 29, 2022

Copy link
Copy Markdown
Collaborator

CI: https://ci.nodejs.org/job/node-test-pull-request/43772/

This was referenced Apr 30, 2022
Open
Open
@3ap

3ap commented May 6, 2022

Copy link
Copy Markdown
ContributorAuthor

Ping? Looks like everything is okay and ready to merge.

@richardlaurichardlau added the commit-queue Add this label to land a pull request using GitHub Actions. label May 6, 2022
@nodejs-github-botnodejs-github-bot removed the commit-queue Add this label to land a pull request using GitHub Actions. label May 6, 2022
@nodejs-github-bot
nodejs-github-bot merged commit 5e9274a into nodejs:masterMay 6, 2022
@nodejs-github-bot

nodejs-github-bot commented May 6, 2022

Copy link
Copy Markdown
Collaborator

Landed in 5e9274a

RafaelGSS pushed a commit that referenced this pull request May 10, 2022
@3ap@RafaelGSS
build: fix indeterminacy of icu_locales value
48f4a71
`icu_locales` is generated by joining values from `set` data structure.
However, `set` doesn't guarantee an order, so the result of
`icu_locales` is not determined. For example, the result value could be
'en,root' or 'root,en'. This fix makes it deterministic.
The main reason of this fix is to restore the reproducibility of the
build because the value of `icu_locales` is embedded into `node` binary.
PR-URL: #42865
Reviewed-By: Richard Lau <rlau@redhat.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Mestery <mestery@protonmail.com>
@RafaelGSSRafaelGSS mentioned this pull request May 10, 2022
Merged
juanarbol pushed a commit that referenced this pull request May 31, 2022
@3ap@juanarbol
build: fix indeterminacy of icu_locales value
e180413
`icu_locales` is generated by joining values from `set` data structure.
However, `set` doesn't guarantee an order, so the result of
`icu_locales` is not determined. For example, the result value could be
'en,root' or 'root,en'. This fix makes it deterministic.
The main reason of this fix is to restore the reproducibility of the
build because the value of `icu_locales` is embedded into `node` binary.
PR-URL: #42865
Reviewed-By: Richard Lau <rlau@redhat.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Mestery <mestery@protonmail.com>
@juanarboljuanarbol mentioned this pull request May 31, 2022
Closed
danielleadams pushed a commit that referenced this pull request Jun 27, 2022
@3ap@danielleadams
build: fix indeterminacy of icu_locales value
69ac672
`icu_locales` is generated by joining values from `set` data structure.
However, `set` doesn't guarantee an order, so the result of
`icu_locales` is not determined. For example, the result value could be
'en,root' or 'root,en'. This fix makes it deterministic.
The main reason of this fix is to restore the reproducibility of the
build because the value of `icu_locales` is embedded into `node` binary.
PR-URL: #42865
Reviewed-By: Richard Lau <rlau@redhat.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Mestery <mestery@protonmail.com>
targos pushed a commit that referenced this pull request Jul 12, 2022
@3ap@targos
build: fix indeterminacy of icu_locales value
f03d5d0
`icu_locales` is generated by joining values from `set` data structure.
However, `set` doesn't guarantee an order, so the result of
`icu_locales` is not determined. For example, the result value could be
'en,root' or 'root,en'. This fix makes it deterministic.
The main reason of this fix is to restore the reproducibility of the
build because the value of `icu_locales` is embedded into `node` binary.
PR-URL: #42865
Reviewed-By: Richard Lau <rlau@redhat.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Mestery <mestery@protonmail.com>
targos pushed a commit that referenced this pull request Jul 31, 2022
@3ap@targos
build: fix indeterminacy of icu_locales value
fba6e96
`icu_locales` is generated by joining values from `set` data structure.
However, `set` doesn't guarantee an order, so the result of
`icu_locales` is not determined. For example, the result value could be
'en,root' or 'root,en'. This fix makes it deterministic.
The main reason of this fix is to restore the reproducibility of the
build because the value of `icu_locales` is embedded into `node` binary.
PR-URL: #42865
Reviewed-By: Richard Lau <rlau@redhat.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Mestery <mestery@protonmail.com>
@targostargos mentioned this pull request Aug 3, 2022
Merged
guangwong pushed a commit to noslate-project/node that referenced this pull request Oct 10, 2022
@3ap@guangwong
build: fix indeterminacy of icu_locales value
958aca0
`icu_locales` is generated by joining values from `set` data structure.
However, `set` doesn't guarantee an order, so the result of
`icu_locales` is not determined. For example, the result value could be
'en,root' or 'root,en'. This fix makes it deterministic.
The main reason of this fix is to restore the reproducibility of the
build because the value of `icu_locales` is embedded into `node` binary.
PR-URL: nodejs/node#42865
Reviewed-By: Richard Lau <rlau@redhat.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Mestery <mestery@protonmail.com>
@3ap
3ap deleted the patch-1 branch October 31, 2022 18:32
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@lpincalpincalpinca approved these changes
@richardlaurichardlaurichardlau approved these changes
+1 more reviewer
@MesteeryMesteeryMesteery approved these changes
Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

buildIssues and PRs related to build files or the CI.needs-ciPRs that need a full CI run.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@3ap@nodejs-github-bot@lpinca@richardlau@Mesteery