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

staging-next 2022-04-10 #168140

Merged
merged 48 commits into from
Apr 12, 2022
Merged

staging-next 2022-04-10 #168140

merged 48 commits into from
Apr 12, 2022

Conversation

vcunat
Copy link
Member

@vcunat vcunat commented Apr 10, 2022

I made this iteration a bit special, focused on security fixes and important bugfixes. And perhaps some other changes that have very low breaking potential. The purpose is to try making this iteration very fast, without requiring human work to progress, so that we don't block some notable problems:


workflow docs: https://nixos.org/manual/nixpkgs/unstable/#submitting-changes-commit-policy
constitutents: https://hydra.nixos.org/job/nixpkgs/staging-next/unstable#tabs-constituents
jobset: https://hydra.nixos.org/jobset/nixpkgs/staging-next

Previous staging-next: #165406

trofi and others added 30 commits March 23, 2022 08:46
Without the change build fails as:

    audit_wrap.c:5010:15: error: invalid use of flexible array member
     5010 |     arg1->buf = (char [])(char *)memcpy(malloc((size)*sizeof(char)), (const char *)(arg2), sizeof(char)*(size));
          |               ^
Signed-off-by: Anders Kaseorg <[email protected]>
Co-authored-by: Guillaume Girol <[email protected]>
powerdns: fix 32-bit builds against glibc
Apply patch that already has been applied upstream:
- madler/zlib#607
- madler/zlib@05796d3

(cherry picked from commit f091c0e)
...fix path to tests, update homepage away from their tract

(cherry picked from commit a8b0e20 from PR #165414)
@vcunat
Copy link
Member Author

vcunat commented Apr 10, 2022

I somehow chose the changes by hand, selecting all corresponding PRs with "security" label and then passing through all the merges in master..staging and picking some more. You can suggest changes to that as well, though we surely don't want to slow this iteration down too much.

The powerdns commit got left in staging-next by accident; it was intended for the previous iteration but got merged too late. It could come directly to master now, but... now I didn't have sufficient motivation to act on it.

@K900
Copy link
Contributor

K900 commented Apr 10, 2022

#166335 should probably go in

github-actions bot and others added 3 commits April 10, 2022 18:01
This partially reverts commit 7f8c513 from PR #168154,
and it's also the same as commit 32f4270.
The tests worked after curl fix (commit 0fad2b3 from PR #167993).
@vcunat
Copy link
Member Author

vcunat commented Apr 10, 2022

TL;DR: so that pycurl problem should be resolved now, both on master and also in a better way here on staging-next.

@fabianhjr
Copy link
Member

fabianhjr commented Apr 10, 2022

made this iteration a bit special, focused on security fixes and important bugfixes.

You can suggest changes to that as well, though we surely don't want to slow this iteration down too much.

Hi @vcunat, just to clarify. The idea is to merge this before the scheduled breaking changes freeze of staging/staging-next on April 17 and then open a new staging-next cycle with the pending staging commits from this cycle?

@vcunat
Copy link
Member Author

vcunat commented Apr 10, 2022

April 17 is the first time restrictions apply for changes merging to staging(-next) branch. Commits that are there now certainly will reach 22.05. (Well of course, it can happen that some of them will be found out to cause too much trouble and reverted, as usual.)

The release schedule itself even plans for two week-long staging(-next) iterations, though historically we typically struggled to make them that fast, so that part might happen a bit differently than planned.

@vcunat vcunat merged commit e111881 into master Apr 12, 2022
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.