Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 73067 - baselayout net module "ipppd" needs patch to work with isdn4k-utils-3.5_pre20041110
Summary: baselayout net module "ipppd" needs patch to work with isdn4k-utils-3.5_pre20...
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: [OLD] baselayout (show other bugs)
Hardware: All Linux
: High normal (vote)
Assignee: Gentoo's Team for Core System packages
URL:
Whiteboard:
Keywords: InVCS
Depends on:
Blocks: 66472
  Show dependency tree
 
Reported: 2004-12-01 13:04 UTC by Martin Wegner
Modified: 2005-01-30 23:58 UTC (History)
3 users (show)

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


Attachments
The patch announced (ipppd-isdn4k-utils-3.5_pre20041110.patch,1.07 KB, patch)
2004-12-01 13:16 UTC, Martin Wegner
Details | Diff
ipppd module to fix stopping (ipppd,2.37 KB, text/plain)
2005-01-12 07:40 UTC, Roy Marples (RETIRED)
Details
ipppd module to fix shutting down (ipppd,2.35 KB, text/plain)
2005-01-13 03:54 UTC, Roy Marples (RETIRED)
Details
fixes clean_pidfile function (functions.patch,654 bytes, patch)
2005-01-13 03:56 UTC, Roy Marples (RETIRED)
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Martin Wegner 2004-12-01 13:04:51 UTC
Since the location of the ipppd binary changed from /sbin to /usr/sbin .

Patch will follow.

Apart from that the routine ipppd_stop is always failing since the last test

test -s "${pidfile}" && false

always fails under normal circumstances so that

eend $? || return $?

always returns 1 and therefore an error.

Reproducible: Always
Steps to Reproduce:
1. emerge =isdn4k-utils-3.5_pre20041110
2. /etc/init.d/net.ippp0 restart
3.

Actual Results:  
States that isdn4k-utils are missing.

Expected Results:  
Find and start ipppd.

$ emerge info
Portage 2.0.51-r3 (default-linux/x86/2004.3, gcc-3.4.3, glibc-2.3.4.20041102-r0,
2.6.9-ck3 i686)
=================================================================
System uname: 2.6.9-ck3 i686 AMD Duron(tm)
Gentoo Base System version 1.6.6
distcc 2.18.2 i686-pc-linux-gnu (protocols 1 and 2) (default port 3632) [disabled]
Autoconf: sys-devel/autoconf-2.59-r5
Automake: sys-devel/automake-1.8.5-r1
Binutils: sys-devel/binutils-2.15.92.0.2-r1
Headers:  sys-kernel/linux-headers-2.4.19,sys-kernel/linux-headers-2.4.22
Libtools: sys-devel/libtool-1.5.2-r7
ACCEPT_KEYWORDS="x86 ~x86"
AUTOCLEAN="yes"
CFLAGS="-march=i686 -O3 -pipe -mmmx -msse -m3dnow -mfpmath=sse -ftracer"
CHOST="i686-pc-linux-gnu"
COMPILER=""
CONFIG_PROTECT="/etc /usr/X11R6/lib/X11/xkb /usr/kde/2/share/config
/usr/kde/3.2/share/config /usr/kde/3.3/env /usr/kde/3.3/share/config
/usr/kde/3.3/shutdown /usr/kde/3/share/config /usr/share/config
/usr/share/texmf/dvipdfm/config/ /usr/share/texmf/dvips/config/
/usr/share/texmf/tex/generic/config/ /usr/share/texmf/tex/platex/config/
/usr/share/texmf/xdvi/ /var/qmail/control"
CONFIG_PROTECT_MASK="/etc/gconf /etc/terminfo /etc/env.d"
CXXFLAGS="-march=i686 -O3 -pipe -mmmx -msse -m3dnow -mfpmath=sse -ftracer"
DISTDIR="/usr/portage/distfiles"
FEATURES="autoaddcvs autoconfig ccache distlocks sandbox sfperms"
GENTOO_MIRRORS="ftp://mirror.switch.ch/mirror/gentoo/
http://ftp-stud.fht-esslingen.de/pub/Mirrors/gentoo/ http://gentoo.osuosl.org
http://www.ibiblio.org/pub/Linux/distributions/gentoo"
MAKEOPTS="-j2"
PKGDIR="/usr/portage/packages"
PORTAGE_TMPDIR="/var/tmp"
PORTDIR="/usr/portage"
PORTDIR_OVERLAY="/usr/local/portage"
SYNC="rsync://rsync.gentoo.org/gentoo-portage"
USE="3dnow 3dnowex X aalib acl acpi acpi4linux alsa apache2 apm ared avi berkdb
bitmap-fonts blender-game bonobo bootspla bzlib cdparanoia cdr clanVoice crypt
cups dba dedicated dga divx4linux doc dvd encode escreen fam fbcon flac
foomaticdb fortran freetype gd gdbm gif gnome gpm gtk gtk2 imagemagick imlib
java jpeg libg++ libwww mad mailwrapper mbox mikmod mmx mmx2 motif mpeg mysql
ncurses net nls oggvorbis opengl oss pam pdflib perl png python quicktime radeon
readline rtc ruby samba sdl session silverxp slang sockets spell spl sse ssl
stencil-buffer svga tcltk tcpd tetex tiff tokenizer truetype usb userlocales
videos wxwindows x86 xchattext xfs xml xml2 xmms xpm xprint xsl xv zlib"
Comment 1 Martin Wegner 2004-12-01 13:16:42 UTC
Created attachment 45078 [details, diff]
The patch announced

This patch should work with all versions of isdn4k-utils
Comment 2 Martin Wegner 2004-12-01 13:17:56 UTC
Forgot to add: for the issue with the test command the patch provides no solution since I do not know one ...
Comment 3 Daniel Black (RETIRED) gentoo-dev 2004-12-02 04:49:07 UTC
net-dialup/isdn4k-utils fixed
Comment 4 Stefan Briesenick (RETIRED) gentoo-dev 2004-12-02 10:55:22 UTC
wuahhh, sorry, but /usr/sbin was the correct place. A symlink would be ok, but moving it into /sbin was silly. All other Distris install it into /usr/sbin.

So not isdn4k-utils had to be fixed, but the baselayout. For the time being, a simple symlink would be the best solution!
Comment 5 Stefan Briesenick (RETIRED) gentoo-dev 2004-12-02 10:58:12 UTC
# ippdp.init script for the ebuild thinks ipppd should be in /sbin
dodir /sbin
mv ${D}/usr/sbin/ipppd ${D}/sbin/ipppd

should be:

# ippdp.init script for the ebuild thinks ipppd should be in /sbin
dodir /sbin
dosym /usr/sbin/ipppd /sbin/ipppd

or just fixing the net.ippp0 script!
Comment 6 Alin Năstac (RETIRED) gentoo-dev 2004-12-03 08:30:19 UTC
I don't understand what base-system has to do with this bug. It is certainly a problem of ipppd.init script but this doesn't belong to baselayout, being a script installed by net-dialup/isdn4k-utils
Comment 7 Alin Năstac (RETIRED) gentoo-dev 2004-12-03 08:55:29 UTC
Fixed in the spirit of comment #4 (fixed path in net.ippp0 script).
I believe Stefan is right; if pppd gets installed in /usr/sbin, why not ipppd do the same, especially because make install does that.

comment #1: I don't have any idea from which script you took those lines. isdn4k-utils doesn't install a executable/script or script which contain the function called ipppd_stop.
Also didn't saw "eend $? || return $?" anywhere.
Comment 8 Martin Wegner 2004-12-03 10:17:03 UTC
In comment #1 I referred to the file

/lib/rcscripts/net.modules.d/ipppd

introduced in baselayout-1.11.6 by uberlord@gentoo.org as requested by my bug #69035 .

I wanted _one_ net script to do everything (as it is the sense of the new modules in the net scripts) and I did not want one net script for each type of interface (isdn, wlan, ...).

So I guess the bug with the "test" issue reported also in comment #1 is not solved thus I'm reopening the bug.
Comment 9 Alin Năstac (RETIRED) gentoo-dev 2004-12-03 10:33:49 UTC
then it is indeed base-system's bug. I will add for the time being a symlink /sbin/ipppd
Comment 10 Stefan Briesenick (RETIRED) gentoo-dev 2004-12-03 10:41:10 UTC
@Alin: perfect! now I'm happy again! ;-)
Comment 11 Stefan Briesenick (RETIRED) gentoo-dev 2004-12-03 10:42:29 UTC
oh, before I forget it. please add a comment to the symlink thingy, that this is a temporary fix (refering to this bug #) and should be removed ASAP.
Comment 12 Stefan Briesenick (RETIRED) gentoo-dev 2004-12-03 10:47:10 UTC
oh, it's already in portage. doesn't matter. just add the comment and don't fix the version or let it be at all.
Comment 13 Roy Marples (RETIRED) gentoo-dev 2004-12-03 14:52:59 UTC
I've fixed the location to /usr/sbin/ipppd in the net-scripts ipppd module

Will be in baselayout-1.11.7
Comment 14 Roy Marples (RETIRED) gentoo-dev 2004-12-04 03:51:03 UTC
Fixed in baselayout-1.11.7-r1
Comment 15 Stefan Briesenick (RETIRED) gentoo-dev 2004-12-04 05:34:52 UTC
ok, then we can remove the symlink in a few weeks. I'm working on some other enhancements for isdn4k-utils, so when they're ready, we can remove it then.

but one question: if the functionality of net.ippp0 is included in net.lo now, then we don't need it (net.ippp0) anymore? Would be great!

btw: is there any gentoo-wiki or such, where I can write a small doc/howto for Gentoo + ISDN/CAPI?
Comment 16 Roy Marples (RETIRED) gentoo-dev 2004-12-04 07:42:26 UTC
net.ipppd will still be required until baselayout-1.11.x or better goes stable

There's a wiki at http://gentoo-wiki.com/Main_Page - but it's not official afaik
Comment 17 Martin Wegner 2005-01-10 12:48:23 UTC
Bug with ipppd module ("test" issue in ipppd_stop in net module ipppd) I referred to in Comment #1 is still not fixed in baselayout-1.11.8:

$ /etc/init.d/net.ippp0 stop
 * Stopping ippp0
 *    Bringing down ippp0
 *       Stopping ipppd for ippp0                                         [ !! ]
 *       Shutting down ippp0 ...                                          [ ok ]

At shutdown I get:

 * Stopping ippp0
 *    Bringing down ippp0
 *       Stopping ipppd for ippp0                                         [ ok ]
 *       Shutting down ippp0 ...                                          [ ok ]

I guess this happens because init already killed ipppd?

So I still think that the error indicated by "!!" when stopping the interface manually is caused by the last test since you use $? as return value for eend
Comment 18 Stefan Briesenick (RETIRED) gentoo-dev 2005-01-10 13:25:52 UTC
huh, well.

correct startup order:

1. /etc/init.d/capi (needed if you have a CAPI based card)
   /etc/init.d/hisax (needed if you have a hisax card)

2. /etc/init.d/isdn (needed for modem-emulation and ippp-devices)

3. /etc/init.d/net.ippp0 (needed only for non-CAPI cards. otherwise use pppd-capiplugin).

stopping order reversed. but all components are optional! So you can't put "need" in depend(). "use" seems to start all optional components in the right order, but stops CAPI before ISDN, which is really odd. Changing "use" to "need" in depend() and everything works fine. But then all optional components are always started, no matter if I want this or not. :-/

Comment 19 Stefan Schweizer (RETIRED) gentoo-dev 2005-01-10 13:47:41 UTC
What does the "Stopping ipppd" problem have to do with the starting order?
Comment 20 Stefan Briesenick (RETIRED) gentoo-dev 2005-01-10 15:39:16 UTC
nothing. but this is another issue we should take care of.
Comment 21 Roy Marples (RETIRED) gentoo-dev 2005-01-12 07:40:27 UTC
Created attachment 48301 [details]
ipppd module to fix stopping

Please test this replacement ipppd module for stopping ipppd
Comment 22 Martin Wegner 2005-01-12 08:03:23 UTC
Attached "ipppd" module works perfect for me
Comment 23 Roy Marples (RETIRED) gentoo-dev 2005-01-12 08:06:39 UTC
Could you clarify that it starts, works, stops and no ipppd process is still running?

And that it can start cleanly and work again?

If you can answer yes to the above, I'll commit to CVS right away.
Sorry for the hassle, but I have no way to actually test the ipppd module :/
Comment 24 Martin Wegner 2005-01-12 09:03:15 UTC
I can confirm every point: it starts, works and stops. ipppd process is correctly started and stopped. It starts cleanly again.
Comment 25 Roy Marples (RETIRED) gentoo-dev 2005-01-12 09:12:26 UTC
Commited to CVS - will be in baselayout-1.11.9
Comment 26 Martin Wegner 2005-01-13 00:05:47 UTC
Sry, but I've just encountered an error that I could not encounter yesterday since I do not often shutdown my computer:

When shutting down during the stopping an error message appears that starts with

ERROR: List of process IDs must follow -p
[...]

The message is longer than this but I could not keep more. This does not appear when stopping the device manually. I cannot say which line exactly causes this.
Comment 27 Roy Marples (RETIRED) gentoo-dev 2005-01-13 03:54:22 UTC
Created attachment 48386 [details]
ipppd module to fix shutting down
Comment 28 Roy Marples (RETIRED) gentoo-dev 2005-01-13 03:56:12 UTC
Created attachment 48387 [details, diff]
fixes clean_pidfile function

Please test with the new module and this patch to functions in
/lib/rcscripts/net.modules.d/helpers.d/functions
Comment 29 Martin Wegner 2005-01-13 23:39:28 UTC
New ipppd module and patch work great for me in all of the above cases.
Comment 30 Roy Marples (RETIRED) gentoo-dev 2005-01-13 23:48:49 UTC
Comitted to CVS - hopefully for the last time ;)
Comment 31 Roy Marples (RETIRED) gentoo-dev 2005-01-30 23:58:41 UTC
Fixed by baselayout-1.11.9