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
Fix udev rule filename(s) #180
base: master
Are you sure you want to change the base?
Conversation
The udev rule filename must match the package name for dh_installudev to recognize it.
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.
I don't have a package with udev rules to test this change against but I had one comment based on my experience adding Suite3 in #147.
link_func(fname, | ||
os.path.join(debian_dir, '%s.udev' % debinfo.package)) | ||
if debinfo.with_python2: | ||
link_func(fname, | ||
os.path.join(debian_dir, '%s.udev' % debinfo.package)) | ||
if debinfo.with_python3: | ||
link_func(fname, | ||
os.path.join(debian_dir, '%s.udev' % debinfo.package3)) |
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.
Have you checked how this will behave when both with_python2 and with_python3 are true? Does it create issues if the udev rules are linked under both package names?
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.
I have tried this and it works fine; the udev rule file actually has to be present twice (with prefix python-
and python3-
) in that case to be picked up by both the Python 2 and 3 package.
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.
I have not given this a try myself but I'm satisfied with the explanation given for the question I had regarding building with both python2 and python3 and the change looks good otherwise.
Can this be merged then? It would be great to have an stdeb bugfix release since this blocks linux-automation/usbsdmux#37 from being merged. |
The udev rule filename must match the package name for dh_installudev to
recognize it.