Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

xkbctrl: sort hash keys #18

Merged
merged 1 commit into from Oct 1, 2018
Merged

xkbctrl: sort hash keys #18

merged 1 commit into from Oct 1, 2018

Conversation

bmwiedemann
Copy link
Contributor

in order to generate reproducible output.
This helps the live-langset-data noarch package
to produce identical package build results for i586 and x86_64

Without this patch, it differed thus:
/usr/share/langset/zu_ZA:
CONSOLE_ENCODING='UTF-8'
-Apply='-option terminate:ctrl_alt_bksp -layout us -model pc105+inet'
+Apply='-model pc105+inet -layout us -option terminate:ctrl_alt_bksp'
XkbLayout='us'

See https://bugzilla.opensuse.org/show_bug.cgi?id=1109534
and https://reproducible-builds.org/ for why this is matters.

@imobachgs
Copy link

It looks good. Please, bump the version and update the changes file.

in order to generate reproducible output.
This helps the live-langset-data noarch package
to produce identical package build results for i586 and x86_64

Without this patch, it differed thus:
/usr/share/langset/zu_ZA:
 CONSOLE_ENCODING='UTF-8'
-Apply='-option terminate:ctrl_alt_bksp -layout us -model pc105+inet'
+Apply='-model pc105+inet -layout us -option terminate:ctrl_alt_bksp'
 XkbLayout='us'

See https://bugzilla.opensuse.org/show_bug.cgi?id=1109534
and https://reproducible-builds.org/ for why this is matters.
@jreidinger jreidinger merged commit 186bb64 into yast:master Oct 1, 2018
@yast-bot
Copy link

yast-bot commented Oct 1, 2018

✔️ Public Jenkins job #3 successfully finished
✔️ Created OBS submit request #639384

@yast-bot
Copy link

yast-bot commented Oct 1, 2018

✔️ Internal Jenkins job #4 successfully finished
✔️ Created IBS submit request #173235

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants