Looks ok to me from a Feature Freeze perspective (but it would be nice
to have an answer to Yuan-Chen's concern, from a technical perspective).

Aside, on ubuntu-drivers-common:

+Files: share/hybrid/json-parser/*
+Copyright: Copyright (c) 2012, 2013 James McLaughlin et al.
+License: BSD
+

Please don't use 'BSD' as a license descriptor, it's ambiguous.
https://www.debian.org/doc/packaging-manuals/copyright-format/1.0
/#license-specification

(also, is there a reason you're embedding a json parser instead of
building against json-c which is supported in main?)

On nvidia-prime:

> +        self._nvidia_runtimepm_file = '/lib/modprobe.d/nvidia-
runtimepm.conf'

Why are you managing files under /lib instead of under /etc? (this is a
preexisting pattern so not a blocker, but it would be nice if this could
be corrected at some point)


** Changed in: nvidia-prime (Ubuntu)
       Status: New => Triaged

** Changed in: nvidia-graphics-drivers-450 (Ubuntu)
       Status: New => Triaged

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/1895855

Title:
  FFE: support for NVIDIA runtimepm (hybrid gfx)

To manage notifications about this bug go to:
https://bugs.launchpad.net/oem-priority/+bug/1895855/+subscriptions

-- 
ubuntu-bugs mailing list
[email protected]
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs

Reply via email to