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

glibc: 2.35-163 -> 2.35-218 #201805

Closed
wants to merge 1 commit into from
Closed

glibc: 2.35-163 -> 2.35-218 #201805

wants to merge 1 commit into from

Conversation

trofi
Copy link
Contributor

@trofi trofi commented Nov 18, 2022

Among other things backport fix for gnu make-4.4 support:
https://sourceware.org/git/?p=glibc.git;a=commitdiff;h=2d7ed98add14f75041499ac189696c9bd3d757fe

Description of changes
Things done
  • Built on platform(s)
    • x86_64-linux
    • aarch64-linux
    • x86_64-darwin
    • aarch64-darwin
  • For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
  • Tested, as applicable:
  • Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
  • Tested basic functionality of all binary files (usually in ./result/bin/)
  • 22.11 Release Notes (or backporting 22.05 Release notes)
    • (Package updates) Added a release notes entry if the change is major or breaking
    • (Module updates) Added a release notes entry if the change is significant
    • (Module addition) Added a release notes entry if adding a new NixOS module
    • (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes
  • Fits CONTRIBUTING.md.

@trofi
Copy link
Contributor Author

trofi commented Nov 19, 2022

Potentially breaks a few timezone-related tests like python310Packages.twisted:

twisted.trial.unittest.FailTest: '2006-06-29T23:00:00+0000' != '2006-06-30T00:00:00+0000'
- 2006-06-29T23:00:00+0000
?         ^^ ^^
+ 2006-06-30T00:00:00+0000
?         ^^ ^^

trofi added a commit to trofi/nixpkgs that referenced this pull request Nov 19, 2022
Not updating the whole snapshot as unrelated patches break some of
timezone-related tests:
    NixOS#201805 (comment)

Let's unblock `make-4.4` first.
@trofi trofi mentioned this pull request Nov 19, 2022
13 tasks
@trofi
Copy link
Contributor Author

trofi commented Nov 19, 2022

Abandoning in favour of less invasive #201936

@vcunat
Copy link
Member

vcunat commented Dec 8, 2022

So, I've been unable to find any reference to the regression in .twisted tests. It's triggered by glibc commit
https://sourceware.org/git/?p=glibc.git;a=commit;h=f6d4c1ac648c4ff8baad94500df0bb13108a4f79
(bisected, easy to revert atop latest 2.35-224)

Full failure log, for reference:

===============================================================================
[FAIL]
Traceback (most recent call last):
  File "/nix/store/l2dwsswg8dd2ybqpxjy3a1iqsf5lj9d5-python3.10-twisted-22.10.0/lib/python3.10/site-packages/tw
isted/logger/test/test_format.py", line 251, in test_formatTimeDefault
    self.assertEqual(
  File "/nix/store/l2dwsswg8dd2ybqpxjy3a1iqsf5lj9d5-python3.10-twisted-22.10.0/lib/python3.10/site-packages/tw
isted/trial/_synctest.py", line 441, in assertEqual
    super().assertEqual(first, second, msg)
  File "/nix/store/xwz1n452s7913wj1338514lqzajmbgla-python3-3.10.8/lib/python3.10/unittest/case.py", line 845,
 in assertEqual
    assertion_func(first, second, msg=msg)
  File "/nix/store/xwz1n452s7913wj1338514lqzajmbgla-python3-3.10.8/lib/python3.10/unittest/case.py", line 1226
, in assertMultiLineEqual
    self.fail(self._formatMessage(msg, standardMsg))
twisted.trial.unittest.FailTest: '2013-09-24T10:40:47+0000 [-#-] XYZZY\n' != '2013-09-24T11:40:47+0000 [-#-] XYZZY\n'
- 2013-09-24T10:40:47+0000 [-#-] XYZZY
?             ^
+ 2013-09-24T11:40:47+0000 [-#-] XYZZY
?             ^


twisted.logger.test.test_format.ClassicLogFormattingTests.test_formatTimeDefault
===============================================================================
[FAIL]
Traceback (most recent call last):
  File "/nix/store/l2dwsswg8dd2ybqpxjy3a1iqsf5lj9d5-python3.10-twisted-22.10.0/lib/python3.10/site-packages/twisted/logger/test/test_format.py", line 589, in test_eventAsTextTimestampOnly
    self.assertEqual(
  File "/nix/store/l2dwsswg8dd2ybqpxjy3a1iqsf5lj9d5-python3.10-twisted-22.10.0/lib/python3.10/site-packages/twisted/trial/_synctest.py", line 441, in assertEqual
    super().assertEqual(first, second, msg)
  File "/nix/store/xwz1n452s7913wj1338514lqzajmbgla-python3-3.10.8/lib/python3.10/unittest/case.py", line 845, in assertEqual
    assertion_func(first, second, msg=msg)
  File "/nix/store/xwz1n452s7913wj1338514lqzajmbgla-python3-3.10.8/lib/python3.10/unittest/case.py", line 1226, in assertMultiLineEqual
    self.fail(self._formatMessage(msg, standardMsg))
twisted.trial.unittest.FailTest: '2013-09-24T10:40:47+0000 ABCD' != '2013-09-24T11:40:47+0000 ABCD'
- 2013-09-24T10:40:47+0000 ABCD
?             ^
+ 2013-09-24T11:40:47+0000 ABCD
?             ^


twisted.logger.test.test_format.EventAsTextTests.test_eventAsTextTimestampOnly
===============================================================================
[FAIL]
Traceback (most recent call last):
  File "/nix/store/l2dwsswg8dd2ybqpxjy3a1iqsf5lj9d5-python3.10-twisted-22.10.0/lib/python3.10/site-packages/twisted/logger/test/test_format.py", line 176, in test_formatTimeWithDefaultFormat
    testForTimeZone(
  File "/nix/store/l2dwsswg8dd2ybqpxjy3a1iqsf5lj9d5-python3.10-twisted-22.10.0/lib/python3.10/site-packages/twisted/logger/test/test_format.py", line 172, in testForTimeZone
    self.assertEqual(formatTime(localDST), expectedDST)
  File "/nix/store/l2dwsswg8dd2ybqpxjy3a1iqsf5lj9d5-python3.10-twisted-22.10.0/lib/python3.10/site-packages/twisted/trial/_synctest.py", line 441, in assertEqual
    super().assertEqual(first, second, msg)
  File "/nix/store/xwz1n452s7913wj1338514lqzajmbgla-python3-3.10.8/lib/python3.10/unittest/case.py", line 845, in assertEqual
    assertion_func(first, second, msg=msg)
  File "/nix/store/xwz1n452s7913wj1338514lqzajmbgla-python3-3.10.8/lib/python3.10/unittest/case.py", line 1226, in assertMultiLineEqual
    self.fail(self._formatMessage(msg, standardMsg))
twisted.trial.unittest.FailTest: '2006-06-29T23:00:00+0000' != '2006-06-30T00:00:00+0000'
- 2006-06-29T23:00:00+0000
?         ^^ ^^
+ 2006-06-30T00:00:00+0000
?         ^^ ^^


twisted.logger.test.test_format.TimeFormattingTests.test_formatTimeWithDefaultFormat
-------------------------------------------------------------------------------

and the relevant source file:
https://github.com/twisted/twisted/blob/twisted-22.10.0/src/twisted/logger/test/test_format.py

EDIT: I tried e-mailing the author of the glibc patch.

@vcunat
Copy link
Member

vcunat commented Dec 8, 2022

#205133

vcunat added a commit that referenced this pull request Dec 9, 2022
For now.  It causes issues in python3Packages.twisted testsuite,
so maybe it's buggy.  The tests might be buggy instead, but so far
we've had no report of the issues that are to be fixed by the patch.
#201805 (comment)
github-actions bot pushed a commit that referenced this pull request Dec 9, 2022
For now.  It causes issues in python3Packages.twisted testsuite,
so maybe it's buggy.  The tests might be buggy instead, but so far
we've had no report of the issues that are to be fixed by the patch.
#201805 (comment)

(cherry picked from commit 5dffcba)
@vcunat
Copy link
Member

vcunat commented Dec 9, 2022

I forwarded the test issue upstream: twisted/twisted#11786
I'd assume that soon we'll drop the revert and fix or skip these twisted tests instead.

@trofi trofi deleted the glibc-update branch December 9, 2022 11:43
ners pushed a commit to ners/nixpkgs that referenced this pull request Dec 16, 2022
Not updating the whole snapshot as unrelated patches break some of
timezone-related tests:
    NixOS#201805 (comment)

Let's unblock `make-4.4` first.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants