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
pps-tools: init at v1.0.2, enable for chrony, gpsd, ntp #42889
Conversation
|
||
stdenv.mkDerivation rec { | ||
baseName = "pps-tools"; | ||
version = "v1.0.2"; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
v
is a part of rev
, it is not a part of version
.
meta = with stdenv.lib;{ | ||
description = "User-space tools for LinuxPPS"; | ||
homepage = http://linuxpps.org/; | ||
license = licenses.gpl2; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The license is gpl2Plus
.
Could you list yourself as the maintainer?
pkgs/servers/gpsd/default.nix
Outdated
scons prefix="$out" leapfetch=no gpsd_user=${gpsdUser} gpsd_group=${gpsdGroup} \ | ||
systemd=yes udevdir="$out/lib/udev" \ | ||
python_libdir="$out/lib/${python2Packages.python.libPrefix}/site-packages" | ||
export CFLAGS="-I${pps-tools.dev}/include" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Add pps-tools
to buildInputs
instead.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Doesn't seem to be detected in that case.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is a common issue with scons-based projects. Please add this patch for gpsd
then:
--- a/SConstruct
+++ b/SConstruct
@@ -226 +226 @@ import_env = (
-envs = {}
+envs = os.environ
pkgs/top-level/all-packages.nix
Outdated
@@ -21855,5 +21855,6 @@ with pkgs; | |||
yrd = callPackage ../tools/networking/yrd { }; | |||
|
|||
powershell = callPackage ../shells/powershell { }; | |||
pps-tools = callPackage ../development/libraries/pps-tools { }; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is primarily a set of Linux tools, please move it to pkgs/os-specific/linux/
and after powertop
in all-packages.nix
.
5279cbb
to
40ec7a7
Compare
Added scons patch, thanks for the hints. |
Motivation for this change
Enable PPS support
Things done
Packaged pps-tools, enabled in supported apps.
sandbox
innix.conf
on non-NixOS)nix-shell -p nox --run "nox-review wip"
./result/bin/
)nix path-info -S
before and after)