zendo
August 22, 2023, 4:22pm
2577
baduhai
August 23, 2023, 12:26am
2578
1 Like
Has been packaged by one maintainer and reviewed by me, the other maintainer
https://github.com/NixOS/nixpkgs/pull/248486
(I’m still waiting for some kind of auto-merge when the maintainers approve… And even more weirdly, I can’t approve the PR )
nat
August 23, 2023, 10:02pm
2582
joopitz
August 24, 2023, 3:02pm
2585
A relatively simple Python package.
This is my first contribution so I’m not 100% if everything is in order, but I couldn’t find any obvious differences to other Python packages.
https://github.com/NixOS/nixpkgs/pull/244656
bbigras
August 24, 2023, 7:05pm
2586
Greizgh
August 24, 2023, 8:03pm
2587
LeixB
August 24, 2023, 9:35pm
2588
(Merged) The maintainer is currently unable to review, so some outside help would be much appreciated!
https://github.com/NixOS/nixpkgs/pull/242557
R-VdP
August 25, 2023, 5:39pm
2592
I refactored the installer tools module to allow for selectively enabling/disabling and overriding the different installer tools (nixos-rebuild, nixos-version, nixos-enter, etc.).
https://github.com/NixOS/nixpkgs/pull/249092
hulr
August 26, 2023, 5:53pm
2594
https://github.com/NixOS/nixpkgs/pull/241784
NixOS:master
← SuperSamus:bottles-gstreamer
opened 10:35AM - 13 Aug 23 UTC
## Description of changes
Because of the gst_all_1 input, 64 bit GStreamer wa… s sneaking into the 32 bit libs.
Basically:
```nix
{ buildFHSEnv
, gst_all_1 # Because of this input...
}:
buildFHSEnv {
multiPkgs = pkgs: with pkgs; {
# This is both 32 bit and 64 bit
hello
# ...this is target arch only (because the `with pkgs;` doesn't apply in this case)
gst_all_1.gst-plugins-base
};
}
```
This commit removes the `gst_all_1` input to correct that.
Also removes setting `GST_PLUGIN_PATH`, as it doesn't seem necessary anymore.
(That env is the reason why I added the `gst_all_1` input and accidentally causing the issue in the first place. Ugh...)
Closes #207641.
## Things done
- Built on platform(s)
- [x] x86_64-linux
- [ ] aarch64-linux
- [ ] x86_64-darwin
- [ ] aarch64-darwin
- [ ] For non-Linux: Is `sandbox = true` set in `nix.conf`? (See [Nix manual](https://nixos.org/manual/nix/stable/command-ref/conf-file.html))
- [ ] Tested, as applicable:
- [NixOS test(s)](https://nixos.org/manual/nixos/unstable/index.html#sec-nixos-tests) (look inside [nixos/tests](https://github.com/NixOS/nixpkgs/blob/master/nixos/tests))
- and/or [package tests](https://nixos.org/manual/nixpkgs/unstable/#sec-package-tests)
- or, for functions and "core" functionality, tests in [lib/tests](https://github.com/NixOS/nixpkgs/blob/master/lib/tests) or [pkgs/test](https://github.com/NixOS/nixpkgs/blob/master/pkgs/test)
- made sure NixOS tests are [linked](https://nixos.org/manual/nixpkgs/unstable/#ssec-nixos-tests-linking) to the relevant packages
- [ ] 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](https://github.com/Mic92/nixpkgs-review#usage)
- [x] Tested basic functionality of all binary files (usually in `./result/bin/`)
- [23.11 Release Notes](https://github.com/NixOS/nixpkgs/blob/master/nixos/doc/manual/release-notes/rl-2311.section.md) (or backporting [23.05 Release notes](https://github.com/NixOS/nixpkgs/blob/master/nixos/doc/manual/release-notes/rl-2305.section.md))
- [ ] (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
- [x] Fits [CONTRIBUTING.md](https://github.com/NixOS/nixpkgs/blob/master/CONTRIBUTING.md).
<!--
To help with the large amounts of pull requests, we would appreciate your
reviews of other pull requests, especially simple package updates. Just leave a
comment describing what you have tested in the relevant package/service.
Reviewing helps to reduce the average time-to-merge for everyone.
Thanks a lot if you do!
List of open PRs: https://github.com/NixOS/nixpkgs/pulls
Reviewing guidelines: https://nixos.org/manual/nixpkgs/unstable/#chap-reviewing-contributions
-->
NixOS:master
← SuperSamus:heroic-gstreamer
opened 03:05PM - 13 Aug 23 UTC
## Description of changes
They are often needed for GStreamer to work.
While… this may look similar to #215248, that PR assumes that "Prefer System Libraries" is disabled (the default).
This PR is for the case in which "Prefer System Libraries" is enabled.
## Things done
- Built on platform(s)
- [x] x86_64-linux
- [ ] aarch64-linux
- [ ] x86_64-darwin
- [ ] aarch64-darwin
- [ ] For non-Linux: Is `sandbox = true` set in `nix.conf`? (See [Nix manual](https://nixos.org/manual/nix/stable/command-ref/conf-file.html))
- [ ] Tested, as applicable:
- [NixOS test(s)](https://nixos.org/manual/nixos/unstable/index.html#sec-nixos-tests) (look inside [nixos/tests](https://github.com/NixOS/nixpkgs/blob/master/nixos/tests))
- and/or [package tests](https://nixos.org/manual/nixpkgs/unstable/#sec-package-tests)
- or, for functions and "core" functionality, tests in [lib/tests](https://github.com/NixOS/nixpkgs/blob/master/lib/tests) or [pkgs/test](https://github.com/NixOS/nixpkgs/blob/master/pkgs/test)
- made sure NixOS tests are [linked](https://nixos.org/manual/nixpkgs/unstable/#ssec-nixos-tests-linking) to the relevant packages
- [ ] 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](https://github.com/Mic92/nixpkgs-review#usage)
- [x] Tested basic functionality of all binary files (usually in `./result/bin/`)
- [23.11 Release Notes](https://github.com/NixOS/nixpkgs/blob/master/nixos/doc/manual/release-notes/rl-2311.section.md) (or backporting [23.05 Release notes](https://github.com/NixOS/nixpkgs/blob/master/nixos/doc/manual/release-notes/rl-2305.section.md))
- [ ] (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
- [x] Fits [CONTRIBUTING.md](https://github.com/NixOS/nixpkgs/blob/master/CONTRIBUTING.md).
<!--
To help with the large amounts of pull requests, we would appreciate your
reviews of other pull requests, especially simple package updates. Just leave a
comment describing what you have tested in the relevant package/service.
Reviewing helps to reduce the average time-to-merge for everyone.
Thanks a lot if you do!
List of open PRs: https://github.com/NixOS/nixpkgs/pulls
Reviewing guidelines: https://nixos.org/manual/nixpkgs/unstable/#chap-reviewing-contributions
-->
https://github.com/NixOS/nixpkgs/pull/248892
https://github.com/NixOS/nixpkgs/pull/250235