Skip to content

Release Notes on mediatomb changes (to master)#124768

Merged
jonringer merged 1 commit intoNixOS:masterfrom
schmittlauch:mediatombRN
May 28, 2021
Merged

Release Notes on mediatomb changes (to master)#124768
jonringer merged 1 commit intoNixOS:masterfrom
schmittlauch:mediatombRN

Conversation

@schmittlauch
Copy link
Member

Motivation for this change

The changes to the mediatomb module are already in master and release-21.05, but the corresponding release notes are not.
Supersedes and closes #100348 by fixing the manual build.
Backport to release-21.05 follows.

Things done
  • Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
  • Built on platform(s)
    • NixOS
    • macOS
    • other Linux distributions
  • Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
  • Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
  • Tested execution of all binary files (usually in ./result/bin/)
  • Added a release notes entry if the change is major or breaking
  • Fits CONTRIBUTING.md.

@schmittlauch schmittlauch requested a review from ryantm as a code owner May 28, 2021 15:55
@github-actions github-actions bot added 6.topic: nixos Issues or PRs affecting NixOS modules, or package usability issues specific to NixOS 8.has: changelog This PR adds or changes release notes 8.has: documentation This PR adds or changes documentation labels May 28, 2021
@schmittlauch
Copy link
Member Author

/cc @SuperSandro2000 as a reviewer of the original PR

@schmittlauch
Copy link
Member Author

backport: #124770

@dotlambda dotlambda changed the title [21.05] Release Notes on mediatomb changes (to master) Release Notes on mediatomb changes (to master) May 28, 2021
@dotlambda
Copy link
Member

Please squash the commits. Feel free to add Co-authored-by.

@ofborg ofborg bot added 10.rebuild-darwin: 0 This PR does not cause any packages to rebuild on Darwin. 10.rebuild-linux: 1-10 This PR causes between 1 and 10 packages to rebuild on Linux. labels May 28, 2021
@schmittlauch
Copy link
Member Author

TIL: fixup rebasing retains the original author.
done

@dotlambda
Copy link
Member

It should usually look like Co-authored-by: name <name@example.com> but it's up to you I guess.

@schmittlauch
Copy link
Member Author

schmittlauch commented May 28, 2021

@dotlambda thanks for the hint, as my contribution is only fixing typos I'm okay with merging as-is.

@dotlambda
Copy link
Member

It's just hurting my eyes :D

@schmittlauch
Copy link
Member Author

if it's really necessary, I'll do the force-push dance again 🙄

Who needs structured metadata, if you can just expect people to use a certain plain-text formatting :D

Note that it made into 2 entries, one about new options in the first section.
Another in the breaking compatibility section due to the openFirewall option
which changes the behavior.

Co-authored-by: schmittlauch <t.schmittlauch+nixos@orlives.de>
@schmittlauch
Copy link
Member Author

🎉

Copy link
Contributor

@jonringer jonringer left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@schmittlauch schmittlauch deleted the mediatombRN branch May 28, 2021 18:05
@schmittlauch
Copy link
Member Author

@jonringer @dotlambda Please do not forget about the backport #124770, these release notes obviously need to go to release-21.05 as well

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

6.topic: nixos Issues or PRs affecting NixOS modules, or package usability issues specific to NixOS 8.has: changelog This PR adds or changes release notes 8.has: documentation This PR adds or changes documentation 10.rebuild-darwin: 0 This PR does not cause any packages to rebuild on Darwin. 10.rebuild-linux: 1-10 This PR causes between 1 and 10 packages to rebuild on Linux.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants