Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 494442 - net-misc/dahdi-tools-2.8.0 version bump
Summary: net-misc/dahdi-tools-2.8.0 version bump
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: All Linux
: Normal normal (vote)
Assignee: Tony Vroon (RETIRED)
URL:
Whiteboard:
Keywords: EBUILD, PATCH
Depends on:
Blocks:
 
Reported: 2013-12-16 12:45 UTC by Oliver Jaksch
Modified: 2014-01-04 22:21 UTC (History)
1 user (show)

See Also:
Package list:
Runtime testing required: ---


Attachments
dahdi-tools-2.8.0.ebuild (dahdi-tools-2.8.0.ebuild,1.23 KB, text/plain)
2013-12-16 12:45 UTC, Oliver Jaksch
Details
updated patchset (gentoo-dahdi-tools-patchset.tar.bz2,2.07 KB, application/x-bzip)
2013-12-16 12:46 UTC, Oliver Jaksch
Details
correct udev directory (06-udevdir.diff,1002 bytes, patch)
2013-12-17 09:44 UTC, Oliver Jaksch
Details | Diff
dahdi-tools-udev.patch (dahdi-tools-udev.patch,1.52 KB, patch)
2013-12-23 16:08 UTC, Jaco Kroon
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Oliver Jaksch 2013-12-16 12:45:49 UTC
Created attachment 365464 [details]
dahdi-tools-2.8.0.ebuild

Digium released dahdi-tools-2.8.0 with some bugfixes and improvements.
I removed '06-linux-headers-3.7.diff' (obsolete) and rediffed the remaining patches. Compiles fine.
Comment 1 Oliver Jaksch 2013-12-16 12:46:57 UTC
Created attachment 365466 [details]
updated patchset
Comment 2 Jeroen Roovers (RETIRED) gentoo-dev 2013-12-16 12:51:13 UTC
Comment on attachment 365464 [details]
dahdi-tools-2.8.0.ebuild

Same as the old ebuild...
Comment 3 Oliver Jaksch 2013-12-17 09:44:10 UTC
Created attachment 365526 [details, diff]
correct udev directory

The original configure script sets udevs dir to /etc which is not wrong but unattractive - Gentoo complains about this to setup all udev rules to /lib/udev/rules.d so this patch corrects this.
Comment 4 Jaco Kroon 2013-12-23 15:59:01 UTC
Does dahdi-tools now install udev rules?  If so, ACK on the correct UDEV directory support, but probable NACK on the removal of the other patch.  A compile-test without the patch will confirm whether or not we need it, but it'll do no harm just keeping it.  The patch however is incomplete as it doesn't update the rule to set ownership to asterisk:dialout but keeps the default asterisk:asterisk.  I'll attach an updated patch in a moment.
Comment 5 Jaco Kroon 2013-12-23 16:08:05 UTC
Created attachment 365999 [details, diff]
dahdi-tools-udev.patch

Updated patch that also sets the correct ownership for created udev nodes.  There are three patches from dahdi-2.7.0.1 that's influenced this:

01-udevinfo-fallback-not-primary.diff - no longer required - the new install mechanism doesn't require the use of udevadm or udevinfo as far as I can tell.
02-dialout-group-not-asterisk.diff - this is the ownership issue.
05-udev-directory.diff - this fixes the path for installing udev rules.

Chainsaw - I've merged the two patches into one as they both aim to correct udev related things.  I hope this is in order - please let me know if I should split this off.
Comment 6 Tony Vroon (RETIRED) gentoo-dev 2014-01-04 22:21:48 UTC
+*dahdi-tools-2.8.0 (04 Jan 2014)
+
+  04 Jan 2014; Tony Vroon <chainsaw@gentoo.org> +dahdi-tools-2.8.0.ebuild:
+  Version bump, as requested by Oliver Jaksch in bug #494442. Initial rediff by
+  Oliver Jaksch, with some finishing touches by Jaco Kroon.