makedev: MAKEDEV creates incorrect DVB device files

[Debian Bug Importer]

Automatically imported from Debian bug report #263136 http://bugs.debian.org/263136

[Adrian Bunk (bunk-fs)]

severity 263136 grave
thanks

This issue definitely needs investigation before Debian 3.1 .

cu
Adrian

--

       "Is there not promise of rain?" Ling Tan asked suddenly out
        of the darkness. There had been need of rain for many days.
       "Only a promise," Lao Er said.
                                       Pearl S. Buck - Dragon Seed

[Debian Bug Importer (debzilla)]

Automatically imported from Debian bug report #263136 http://bugs.debian.org/263136

[Debian Bug Importer (debzilla)]

Message-ID: <email address hidden>
Date: Tue, 03 Aug 2004 06:49:22 +0300
From: Tuomas Jormola <email address hidden>
To: Debian Bug Tracking System <email address hidden>
Subject: makedev: MAKEDEV creates incorrect DVB device files

Package: makedev
Version: 2.3.1-74
Severity: important

The Debian MAKEDEV creates dvb device files under /dev/dvb with major char
250 but 212 is the correct one. See this ChangeSet snippet from Linus'
bitkeeper tree for more info.

# ChangeSet
# 2004/07/28 22:58:23-07:00 <email address hidden>
# [PATCH] DVB major number
#
# The DVB merge forgot an important change - DVB is currently using
# major 250, which is "local" while devices.txt (and thus FSSTND/LSB)
# require
# it uses the officially allocated 212 major.
#
# Fortunately most 2.6 users don't currently use DVB and vendors ship
# devfs
# or versions of makedev that do the right thing and generate 212
# majors.
#
# Fix below, original author: Red Hat <email address hidden>, OSDL
# certificate
# of authorship included by reference.
#
# Signed-off-by: Andrew Morton <email address hidden>
# Signed-off-by: Linus Torvalds <email address hidden>
#
# drivers/media/dvb/dvb-core/dvbdev.h
# 2004/07/28 21:58:33-07:00 <email address hidden> +1 -1
# DVB major number

br,
Tuomas Jormola <email address hidden>

-- System Information:
Debian Release: 3.1
  APT prefers unstable
  APT policy: (500, 'unstable')
Architecture: i386 (i686)
Kernel: Linux 2.6.7-mm5-udm1
Locale: LANG=C, LC_CTYPE=fi_FI@euro

Versions of packages makedev depends on:
ii base-passwd 3.5.7 Debian base system master password

-- no debconf information

[Debian Bug Importer (debzilla)]

Message-ID: <email address hidden>
Date: Sat, 7 Aug 2004 00:22:06 +0200
From: Adrian Bunk <email address hidden>
To: <email address hidden>
Cc: <email address hidden>
Subject: This issue might break applications

severity 263136 grave
thanks

This issue definitely needs investigation before Debian 3.1 .

cu
Adrian

--

       "Is there not promise of rain?" Ling Tan asked suddenly out
        of the darkness. There had been need of rain for many days.
       "Only a promise," Lao Er said.
                                       Pearl S. Buck - Dragon Seed

[Matt Zimmerman (mdz)]

Warty uses udev

[Bdale Garbee (bdale)]

Adrian Bunk <email address hidden> writes:

> severity 263136 grave

Grave? Surely you jest.

Bdale

[Debian Bug Importer (debzilla)]

Message-ID: <email address hidden>
Date: Fri, 06 Aug 2004 20:24:16 -0600
From: Bdale Garbee <email address hidden>
To: Adrian Bunk <email address hidden>
Cc: <email address hidden>
Subject: Re: Bug#263136: This issue might break applications

Adrian Bunk <email address hidden> writes:

> severity 263136 grave

Grave? Surely you jest.

Bdale

[Adrian Bunk (bunk-fs)]

On Fri, Aug 06, 2004 at 08:24:16PM -0600, Bdale Garbee wrote:
> Adrian Bunk <email address hidden> writes:
>
> > severity 263136 grave
>
> Grave? Surely you jest.

It's an issue that should definitely fixed before Debian 3.1 .

Nowadays, everyone in Debian stares at the RC bug count, but bugs of the
severity important or lower pretty often get ignored (since they don't
affect the magic RC bug count).

According to the latest "freeze starts next week" announcements of your
release managers a "hard freeze" of "base packages" (including makedev)
should start on "31 July" (later changed to "2 August 2004").

Unless Steve Langasek is a liar [1], the only way to get an updated
makedev into sarge today is "allowed through uploads to
testing-proposed-updates with hand-approval by the release team".

> Bdale

cu
Adrian

[1] http://lists.debian.org/debian-devel-announce/2004/08/msg00001.html

--

       "Is there not promise of rain?" Ling Tan asked suddenly out
        of the darkness. There had been need of rain for many days.
       "Only a promise," Lao Er said.
                                       Pearl S. Buck - Dragon Seed

[Debian Bug Importer (debzilla)]

Message-ID: <email address hidden>
Date: Sat, 7 Aug 2004 16:57:55 +0200
From: Adrian Bunk <email address hidden>
To: Bdale Garbee <email address hidden>
Cc: <email address hidden>
Subject: Re: Bug#263136: This issue might break applications

On Fri, Aug 06, 2004 at 08:24:16PM -0600, Bdale Garbee wrote:
> Adrian Bunk <email address hidden> writes:
>
> > severity 263136 grave
>
> Grave? Surely you jest.

It's an issue that should definitely fixed before Debian 3.1 .

Nowadays, everyone in Debian stares at the RC bug count, but bugs of the
severity important or lower pretty often get ignored (since they don't
affect the magic RC bug count).

According to the latest "freeze starts next week" announcements of your
release managers a "hard freeze" of "base packages" (including makedev)
should start on "31 July" (later changed to "2 August 2004").

Unless Steve Langasek is a liar [1], the only way to get an updated
makedev into sarge today is "allowed through uploads to
testing-proposed-updates with hand-approval by the release team".

> Bdale

cu
Adrian

[1] http://lists.debian.org/debian-devel-announce/2004/08/msg00001.html

--

       "Is there not promise of rain?" Ling Tan asked suddenly out
        of the darkness. There had been need of rain for many days.
       "Only a promise," Lao Er said.
                                       Pearl S. Buck - Dragon Seed

[Thomas Schmidt (tschmidt)]

* Adrian Bunk schrieb am 07.08.04, um 16:57 Uhr:
> On Fri, Aug 06, 2004 at 08:24:16PM -0600, Bdale Garbee wrote:
> > Adrian Bunk <email address hidden> writes:
> >
> > > severity 263136 grave
> >
> > Grave? Surely you jest.
>
> It's an issue that should definitely fixed before Debian 3.1 .

Yes, this issue should be discussed, but i do not see a simply way to
solve it.

One reason for keeping major 250 in sarge is that it is allmost sure
that sarge wont ship (at release-time) a kernel, which uses 212, and
the dvb-driver-source-package (for 2.4-users) also uses 250. So
just changing makedev to use would 212 break the device-generation for
most users of sarge. (At least a long as they use a kernel <=2.6.7 or
the dvb-driver-source-package in sarge.)

Ok, there are also reasons to change it, because it is clear that some
sarge-users would like to use a kernel >=2.6.8, but i do not see how
this change can be made without breaking the dvb-device-generation for
the other sarge-users.

Note: I am the (co-)maintainer of the packages linuxtv-dvb,
linuxtv-dvb-apps and vdr, which all rely on the correct
dvb-device-generation, as they use "makedev dvb" in their postinst.

There is also at least one other package, which uses it in its
postinst: dvbtune

Regards,
Thomas

--
Thomas Schmidt
:wq

[Debian Bug Importer (debzilla)]

Message-ID: <20040811083235.GA26899@slipstream>
Date: Wed, 11 Aug 2004 10:32:35 +0200
From: Thomas Schmidt <email address hidden>
To: Adrian Bunk <email address hidden>
Cc: <email address hidden>, <email address hidden>
Subject: Re: Bug#263136: This issue might break applications

--k+w/mQv8wyuph6w0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
Content-Transfer-Encoding: quoted-printable

* Adrian Bunk schrieb am 07.08.04, um 16:57 Uhr:
> On Fri, Aug 06, 2004 at 08:24:16PM -0600, Bdale Garbee wrote:
> > Adrian Bunk <email address hidden> writes:
> >=20
> > > severity 263136 grave
> >=20
> > Grave? Surely you jest.
>=20
> It's an issue that should definitely fixed before Debian 3.1 .

Yes, this issue should be discussed, but i do not see a simply way to=20
solve it.

One reason for keeping major 250 in sarge is that it is allmost sure
that sarge wont ship (at release-time) a kernel, which uses 212, and
the dvb-driver-source-package (for 2.4-users) also uses 250. So
just changing makedev to use would 212 break the device-generation for
most users of sarge. (At least a long as they use a kernel <=3D2.6.7 or
the dvb-driver-source-package in sarge.)

Ok, there are also reasons to change it, because it is clear that some
sarge-users would like to use a kernel >=3D2.6.8, but i do not see how
this change can be made without breaking the dvb-device-generation for
the other sarge-users.

Note: I am the (co-)maintainer of the packages linuxtv-dvb,
linuxtv-dvb-apps and vdr, which all rely on the correct
dvb-device-generation, as they use "makedev dvb" in their postinst.=20

There is also at least one other package, which uses it in its
postinst: dvbtune

Regards,
Thomas

--=20
Thomas Schmidt
:wq

--k+w/mQv8wyuph6w0
Content-Type: application/pgp-signature; name="signature.asc"
Content-Description: Digital signature
Content-Disposition: inline

-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.2.5 (GNU/Linux)

iD8DBQFBGdmjc9+NqwoydlIRAlAGAKCkfBedmW4rLc5jS/NhX1fJ1O9JBgCaAuB/
xbemkzI2D9s1/QRZlSJxdzU=
=BuoY
-----END PGP SIGNATURE-----

--k+w/mQv8wyuph6w0--

[Adrian Bunk (bunk-fs)]

On Wed, Aug 11, 2004 at 10:32:35AM +0200, Thomas Schmidt wrote:
> * Adrian Bunk schrieb am 07.08.04, um 16:57 Uhr:
> > On Fri, Aug 06, 2004 at 08:24:16PM -0600, Bdale Garbee wrote:
> > > Adrian Bunk <email address hidden> writes:
> > >
> > > > severity 263136 grave
> > >
> > > Grave? Surely you jest.
> >
> > It's an issue that should definitely fixed before Debian 3.1 .
>
> Yes, this issue should be discussed, but i do not see a simply way to
> solve it.
>
> One reason for keeping major 250 in sarge is that it is allmost sure
> that sarge wont ship (at release-time) a kernel, which uses 212, and
> the dvb-driver-source-package (for 2.4-users) also uses 250. So
> just changing makedev to use would 212 break the device-generation for
> most users of sarge. (At least a long as they use a kernel <=2.6.7 or
> the dvb-driver-source-package in sarge.)
>
> Ok, there are also reasons to change it, because it is clear that some
> sarge-users would like to use a kernel >=2.6.8, but i do not see how
> this change can be made without breaking the dvb-device-generation for
> the other sarge-users.
>...

I wouldn't call it nice, but what about the following workaround:

Add two dvb targets to MAKEDEV, and add an init script to makedev that
executes the correct one at boot time (depending on the kernel version).

> Regards,
> Thomas

cu
Adrian

--

       "Is there not promise of rain?" Ling Tan asked suddenly out
        of the darkness. There had been need of rain for many days.
       "Only a promise," Lao Er said.
                                       Pearl S. Buck - Dragon Seed

[Debian Bug Importer (debzilla)]

Message-ID: <email address hidden>
Date: Wed, 11 Aug 2004 12:57:09 +0200
From: Adrian Bunk <email address hidden>
To: Thomas Schmidt <email address hidden>
Cc: <email address hidden>, <email address hidden>
Subject: Re: Bug#263136: This issue might break applications

On Wed, Aug 11, 2004 at 10:32:35AM +0200, Thomas Schmidt wrote:
> * Adrian Bunk schrieb am 07.08.04, um 16:57 Uhr:
> > On Fri, Aug 06, 2004 at 08:24:16PM -0600, Bdale Garbee wrote:
> > > Adrian Bunk <email address hidden> writes:
> > >
> > > > severity 263136 grave
> > >
> > > Grave? Surely you jest.
> >
> > It's an issue that should definitely fixed before Debian 3.1 .
>
> Yes, this issue should be discussed, but i do not see a simply way to
> solve it.
>
> One reason for keeping major 250 in sarge is that it is allmost sure
> that sarge wont ship (at release-time) a kernel, which uses 212, and
> the dvb-driver-source-package (for 2.4-users) also uses 250. So
> just changing makedev to use would 212 break the device-generation for
> most users of sarge. (At least a long as they use a kernel <=2.6.7 or
> the dvb-driver-source-package in sarge.)
>
> Ok, there are also reasons to change it, because it is clear that some
> sarge-users would like to use a kernel >=2.6.8, but i do not see how
> this change can be made without breaking the dvb-device-generation for
> the other sarge-users.
>...

I wouldn't call it nice, but what about the following workaround:

Add two dvb targets to MAKEDEV, and add an init script to makedev that
executes the correct one at boot time (depending on the kernel version).

> Regards,
> Thomas

cu
Adrian

--

       "Is there not promise of rain?" Ling Tan asked suddenly out
        of the darkness. There had been need of rain for many days.
       "Only a promise," Lao Er said.
                                       Pearl S. Buck - Dragon Seed

[Thomas Schmidt (tschmidt)]

* Adrian Bunk schrieb am 11.08.04, um 12:57 Uhr:
> I wouldn't call it nice, but what about the following workaround:
>
> Add two dvb targets to MAKEDEV, and add an init script to makedev that
> executes the correct one at boot time (depending on the kernel version).

Yes, this could work, but i would extend/change it a little bit.

I would do the check for the running kernel inside makedev, so that
"MAKEDEV dvb" decides which major-version to use, depending on the
running kernel. (kernel >= 2.6.8-rc3 -> char-major-212, otherwise 250)

Then the init-script (which makedev allready uses) would just need to
call "MAKEDEV dvb" (but only when /dev/dvb/ allready exists).

Regards,
Thomas

--
Thomas Schmidt
<email address hidden>

[Debian Bug Importer (debzilla)]

Message-ID: <20040811173943.GA26419@slipstream>
Date: Wed, 11 Aug 2004 19:39:44 +0200
From: Thomas Schmidt <email address hidden>
To: Adrian Bunk <email address hidden>
Cc: <email address hidden>, <email address hidden>
Subject: Re: Bug#263136: This issue might break applications

--xHFwDpU9dbj6ez1V
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
Content-Transfer-Encoding: quoted-printable

* Adrian Bunk schrieb am 11.08.04, um 12:57 Uhr:
> I wouldn't call it nice, but what about the following workaround:
>=20
> Add two dvb targets to MAKEDEV, and add an init script to makedev that=20
> executes the correct one at boot time (depending on the kernel version).

Yes, this could work, but i would extend/change it a little bit.

I would do the check for the running kernel inside makedev, so that
"MAKEDEV dvb" decides which major-version to use, depending on the
running kernel. (kernel >=3D 2.6.8-rc3 -> char-major-212, otherwise 250)

Then the init-script (which makedev allready uses) would just need to=20
call "MAKEDEV dvb" (but only when /dev/dvb/ allready exists).

Regards,
Thomas

--=20
Thomas Schmidt
<email address hidden>

--xHFwDpU9dbj6ez1V
Content-Type: application/pgp-signature; name="signature.asc"
Content-Description: Digital signature
Content-Disposition: inline

-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.2.5 (GNU/Linux)

iD8DBQFBGlnfc9+NqwoydlIRAoeyAKDEa+wyGFXEAsDD4RPecvbwJPUkrQCgyWBG
pIsunPmw4DFEaqSJU93cIpQ=
=5oOz
-----END PGP SIGNATURE-----

--xHFwDpU9dbj6ez1V--

[Adrian Bunk (bunk-fs)]

On Wed, Aug 11, 2004 at 07:39:44PM +0200, Thomas Schmidt wrote:
> * Adrian Bunk schrieb am 11.08.04, um 12:57 Uhr:
> > I wouldn't call it nice, but what about the following workaround:
> >
> > Add two dvb targets to MAKEDEV, and add an init script to makedev that
> > executes the correct one at boot time (depending on the kernel version).
>
> Yes, this could work, but i would extend/change it a little bit.
>
> I would do the check for the running kernel inside makedev, so that
> "MAKEDEV dvb" decides which major-version to use, depending on the
> running kernel. (kernel >= 2.6.8-rc3 -> char-major-212, otherwise 250)
>...

Agreed.

But parsing will be simpler if it's simply checked for >= 2.6.8 (it
should be enough if all final kernels are correctly supported).

> Regards,
> Thomas

cu
Adrian

--

       "Is there not promise of rain?" Ling Tan asked suddenly out
        of the darkness. There had been need of rain for many days.
       "Only a promise," Lao Er said.
                                       Pearl S. Buck - Dragon Seed

[Debian Bug Importer (debzilla)]

Message-ID: <email address hidden>
Date: Wed, 11 Aug 2004 21:47:27 +0200
From: Adrian Bunk <email address hidden>
To: Thomas Schmidt <email address hidden>
Cc: <email address hidden>, <email address hidden>
Subject: Re: Bug#263136: This issue might break applications

On Wed, Aug 11, 2004 at 07:39:44PM +0200, Thomas Schmidt wrote:
> * Adrian Bunk schrieb am 11.08.04, um 12:57 Uhr:
> > I wouldn't call it nice, but what about the following workaround:
> >
> > Add two dvb targets to MAKEDEV, and add an init script to makedev that
> > executes the correct one at boot time (depending on the kernel version).
>
> Yes, this could work, but i would extend/change it a little bit.
>
> I would do the check for the running kernel inside makedev, so that
> "MAKEDEV dvb" decides which major-version to use, depending on the
> running kernel. (kernel >= 2.6.8-rc3 -> char-major-212, otherwise 250)
>...

Agreed.

But parsing will be simpler if it's simply checked for >= 2.6.8 (it
should be enough if all final kernels are correctly supported).

> Regards,
> Thomas

cu
Adrian

--

       "Is there not promise of rain?" Ling Tan asked suddenly out
        of the darkness. There had been need of rain for many days.
       "Only a promise," Lao Er said.
                                       Pearl S. Buck - Dragon Seed

[Bdale Garbee (bdale)]

Adrian Bunk <email address hidden> writes:

> But parsing will be simpler if it's simply checked for >= 2.6.8 (it
> should be enough if all final kernels are correctly supported).

Someone want to offer me a patch that implements this? It's not hard, but I'm
working on other things today and it'll get done faster if someone hands me a
patch.

There's already code in the consoleonly target to do kernel version checking.

Bdale

[Thomas Schmidt (tschmidt)]

* Bdale Garbee schrieb am 12.08.04, um 23:22 Uhr:
> Someone want to offer me a patch that implements this? It's not hard, but I'm
> working on other things today and it'll get done faster if someone hands me a
> patch.
>
> There's already code in the consoleonly target to do kernel version checking.

Ah, this was the part where i did not know how to realize this check,
thanks for the hint. :)

Well, i created a patch for this bug now, but i am not completely sure
about the check of the $kern_rev3, because it will also output just 8
on kernels like 2.6.8-pre2 (where char-major 250 was used).

The second thing where i am not sure is if it is needed to check for
devfs before calling makedev, and if the ckeck in my patch is right.

See the attached file.

Thomas

--
Thomas Schmidt
<email address hidden>

[Thomas Schmidt (tschmidt)]

tags 263136 + patch

--
Thomas Schmidt
<email address hidden>

[Debian Bug Importer (debzilla)]

Message-ID: <email address hidden>
Date: Thu, 12 Aug 2004 17:22:50 -0400
From: Bdale Garbee <email address hidden>
To: Adrian Bunk <email address hidden>
Cc: Thomas Schmidt <email address hidden>,
 <email address hidden>, <email address hidden>
Subject: Re: Bug#263136: This issue might break applications

Adrian Bunk <email address hidden> writes:

> But parsing will be simpler if it's simply checked for >= 2.6.8 (it
> should be enough if all final kernels are correctly supported).

Someone want to offer me a patch that implements this? It's not hard, but I'm
working on other things today and it'll get done faster if someone hands me a
patch.

There's already code in the consoleonly target to do kernel version checking.

Bdale

[Debian Bug Importer (debzilla)]

Message-ID: <20040812222023.GA18474@slipstream>
Date: Fri, 13 Aug 2004 00:20:23 +0200
From: Thomas Schmidt <email address hidden>
To: Bdale Garbee <email address hidden>
Cc: Adrian Bunk <email address hidden>, <email address hidden>, <email address hidden>
Subject: Re: Bug#263136: This issue might break applications

--mxv5cy4qt+RJ9ypb
Content-Type: multipart/mixed; boundary="wRRV7LY7NUeQGEoC"
Content-Disposition: inline

--wRRV7LY7NUeQGEoC
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
Content-Transfer-Encoding: quoted-printable

* Bdale Garbee schrieb am 12.08.04, um 23:22 Uhr:
> Someone want to offer me a patch that implements this? It's not hard, bu=
t I'm
> working on other things today and it'll get done faster if someone hands =
me a
> patch.
>=20
> There's already code in the consoleonly target to do kernel version check=
ing.

Ah, this was the part where i did not know how to realize this check,
thanks for the hint. :)

Well, i created a patch for this bug now, but i am not completely sure
about the check of the $kern_rev3, because it will also output just 8
on kernels like 2.6.8-pre2 (where char-major 250 was used).

The second thing where i am not sure is if it is needed to check for
devfs before calling makedev, and if the ckeck in my patch is right.

See the attached file.

Thomas

--=20
Thomas Schmidt
<email address hidden>

--wRRV7LY7NUeQGEoC
Content-Type: text/plain; charset=us-ascii
Content-Disposition: attachment; filename="263136.diff"
Content-Transfer-Encoding: quoted-printable

diff -Nurd makedev-2.3.1/debian/init.d makedev-2.3.1.new/debian/init.d
--- makedev-2.3.1/debian/init.d 2004-08-13 00:08:20.000000000 +0200
+++ makedev-2.3.1.new/debian/init.d 2004-08-13 00:10:38.000000000 +0200
@@ -8,6 +8,20 @@
 case "$1" in
   start)
  test -L /dev/MAKEDEV || ln -fs /sbin/MAKEDEV /dev/MAKEDEV
+
+ # create dvb-devices, if the dvb-directory allready exists, and=20
+ # no devfs is used (this a workaround for the changed major-number
+ # of the dvb-devices, introduced in kernel 2.6.8, to be able to=20
+ # switch between kernels <=3D 2.6.7 and >=3D 2.6.8, without manual=20
+ # recreation of the devices (MAKEDEV dvb checks the kernel-version=20
+ # and uses the correct major-number))
+=09
+ if [ ! -e /dev/.devfsd ]; then
+ if [ -d /dev/dvb ]; then
+ cd /dev && ./MAKEDEV dvb
+ fi
+ fi
+=09
  ;;
   stop|reload|restart|force-reload)
  ;;
diff -Nurd makedev-2.3.1/MAKEDEV makedev-2.3.1.new/MAKEDEV
--- makedev-2.3.1/MAKEDEV 2004-08-13 00:08:20.000000000 +0200
+++ makedev-2.3.1.new/MAKEDEV 2004-08-13 00:09:44.000000000 +0200
@@ -1974,18 +1974,32 @@
   done
   ;;
  dvb)
+ # check if kernel-version is >=3D 2.6.8, if yes, create dvb-devices with
+ # major-number 212, in the other case 250
+ =09
+ kern_rev1=3D`uname -r | sed -e 's@^\([^.]*\)\..*@\1@'`
+ kern_rev2=3D`uname -r | sed -e 's@^[^.]*\.\([^.]*\)\..*@\1@'`
+ kern_rev3=3D`uname -r | sed -e 's@^[^.]*\.[^.]*\.\([^.][0-9]*\).*@\1@'`
+ =20
+ dvb_major=3D250
+
+ if [ $kern_rev1 -ge 2 ] && [ $kern_rev2 -ge 6 ] && [ $kern_rev3 -ge 8 ]
+ then
+ dvb_major=3D212
+ fi
+
   mkdir -p dvb
   for i in 0 1 2 3
   do
    mkdir -p dvb/adapter$i...

[Debian Bug Importer (debzilla)]

Message-ID: <20040812222456.GA3451@slipstream>
Date: Fri, 13 Aug 2004 00:24:56 +0200
From: Thomas Schmidt <email address hidden>
To: <email address hidden>
Subject: Re: Bug#263136: This issue might break applications

--UugvWAfsgieZRqgk
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
Content-Transfer-Encoding: quoted-printable

tags 263136 + patch

--=20
Thomas Schmidt
<email address hidden>

--UugvWAfsgieZRqgk
Content-Type: application/pgp-signature; name="signature.asc"
Content-Description: Digital signature
Content-Disposition: inline

-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.2.5 (GNU/Linux)

iD8DBQFBG+44c9+NqwoydlIRAtDOAJ4llPesmsoaEwTZp8mmrpJZA4WxWACgyg1M
bfopNQPYh9H+kACX+VLDjEQ=
=/wjB
-----END PGP SIGNATURE-----

--UugvWAfsgieZRqgk--

[Thomas Schmidt (tschmidt)]

* Thomas Schmidt schrieb am 13.08.04, um 00:20 Uhr:
> See the attached file.

Sorry, the patch was incomplete, it would match only on 2.6.x-kernels,
here is the version that should work with 2.7.x and 3.x also. :)

--
Thomas Schmidt
:wq

[Debian Bug Importer (debzilla)]

Message-ID: <20040813080523.GA16123@slipstream>
Date: Fri, 13 Aug 2004 10:05:23 +0200
From: Thomas Schmidt <email address hidden>
To: Bdale Garbee <email address hidden>
Cc: Adrian Bunk <email address hidden>, <email address hidden>, <email address hidden>
Subject: Re: Bug#263136: This issue might break applications

--24zk1gE8NUlDmwG9
Content-Type: multipart/mixed; boundary="h31gzZEtNLTqOjlF"
Content-Disposition: inline

--h31gzZEtNLTqOjlF
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
Content-Transfer-Encoding: quoted-printable

* Thomas Schmidt schrieb am 13.08.04, um 00:20 Uhr:
> See the attached file.

Sorry, the patch was incomplete, it would match only on 2.6.x-kernels,
here is the version that should work with 2.7.x and 3.x also. :)

--=20
Thomas Schmidt
:wq

--h31gzZEtNLTqOjlF
Content-Type: text/plain; charset=us-ascii
Content-Disposition: attachment; filename="263136.diff"
Content-Transfer-Encoding: quoted-printable

diff -Nurd makedev-2.3.1/debian/init.d makedev-2.3.1.new/debian/init.d
--- makedev-2.3.1/debian/init.d 2004-08-13 00:08:20.000000000 +0200
+++ makedev-2.3.1.new/debian/init.d 2004-08-13 00:10:38.000000000 +0200
@@ -8,6 +8,20 @@
 case "$1" in
   start)
  test -L /dev/MAKEDEV || ln -fs /sbin/MAKEDEV /dev/MAKEDEV
+
+ # create dvb-devices, if the dvb-directory allready exists, and=20
+ # no devfs is used (this a workaround for the changed major-number
+ # of the dvb-devices, introduced in kernel 2.6.8, to be able to=20
+ # switch between kernels <=3D 2.6.7 and >=3D 2.6.8, without manual=20
+ # recreation of the devices (MAKEDEV dvb checks the kernel-version=20
+ # and uses the correct major-number))
+=09
+ if [ ! -e /dev/.devfsd ]; then
+ if [ -d /dev/dvb ]; then
+ cd /dev && ./MAKEDEV dvb
+ fi
+ fi
+=09
  ;;
   stop|reload|restart|force-reload)
  ;;
diff -Nurd makedev-2.3.1/MAKEDEV makedev-2.3.1.new/MAKEDEV
--- makedev-2.3.1/MAKEDEV 2004-08-13 00:08:20.000000000 +0200
+++ makedev-2.3.1.new/MAKEDEV 2004-08-13 10:01:11.000000000 +0200
@@ -1974,18 +1974,33 @@
   done
   ;;
  dvb)
+ # check if kernel-version is >=3D 2.6.8, if yes, create dvb-devices with
+ # major-number 212, in the other case 250
+ =09
+ kern_rev1=3D`uname -r | sed -e 's@^\([^.]*\)\..*@\1@'`
+ kern_rev2=3D`uname -r | sed -e 's@^[^.]*\.\([^.]*\)\..*@\1@'`
+ kern_rev3=3D`uname -r | sed -e 's@^[^.]*\.[^.]*\.\([^.][0-9]*\).*@\1@'`
+ =20
+ dvb_major=3D250
+
+ if [ $kern_rev1 -gt 2 ] || ([ $kern_rev1 -eq 2 ] && [ $kern_rev2 -gt 6 ]=
) \
+ || ([ $kern_rev1 -eq 2 ] && [ $kern_rev2 -eq 6 ] && [ $kern_rev3 -ge 8 =
])
+ then=20
+ dvb_major=3D212=20
+ fi
+
   mkdir -p dvb
   for i in 0 1 2 3
   do
    mkdir -p dvb/adapter$i
- makedev dvb/adapter$i/video0 c 250 `math 64 \* $i + 0` $video
- makedev dvb/adapter$i/audio0 c 250 `math 64 \* $i + 1` $video
- makedev dvb/adapter$i/frontend0 c 250 `math 64 \* $i + 3` $video
- makedev dvb/adapter$i/demux0 c 250 `math 64 \* $i + 4` $video
- makedev dvb/adapter$i/dvr0 c 250 `math 64 \* $i + 5` $video
- makedev dvb/adapter$i/ca0 c 250 `math 64 \* $i + 6` $video
- makedev dvb/adapter$i/net0 c 250 `math 64 \* $i + 7` $video
- makedev dvb/adapter$i/...

[Bdale Garbee (bdale)]

tag 263136 +pending
thanks

I've applied the patch in my revision control system for the next upload.

Bdale

[Bdale Garbee (bdale)]

Source: makedev
Source-Version: 2.3.1-75

We believe that the bug you reported is fixed in the latest version of
makedev, which is due to be installed in the Debian FTP archive:

makedev_2.3.1-75.diff.gz
  to pool/main/m/makedev/makedev_2.3.1-75.diff.gz
makedev_2.3.1-75.dsc
  to pool/main/m/makedev/makedev_2.3.1-75.dsc
makedev_2.3.1-75_all.deb
  to pool/main/m/makedev/makedev_2.3.1-75_all.deb

A summary of the changes between this version and the previous one is
attached.

Thank you for reporting the bug, which will now be closed. If you
have further comments please address them to <email address hidden>,
and the maintainer will reopen the bug report if appropriate.

Debian distribution maintenance software
pp.
Bdale Garbee <email address hidden> (supplier of updated makedev package)

(This message was generated automatically at their request; if you
believe that there is a problem with it please contact the archive
administrators by mailing <email address hidden>)

-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

Format: 1.7
Date: Wed, 18 Aug 2004 12:42:40 -0600
Source: makedev
Binary: makedev
Architecture: source all
Version: 2.3.1-75
Distribution: unstable
Urgency: low
Maintainer: Bdale Garbee <email address hidden>
Changed-By: Bdale Garbee <email address hidden>
Description:
 makedev - Creates device files in /dev
Closes: 261477 263136
Changes:
 makedev (2.3.1-75) unstable; urgency=low
 .
   * fix one more POSIX compliance issue in postinst, closes: #261477
   * patch from Thomas Schmidt to fix problems in handling DVB devices,
     which change major number at kernel 2.6.8, closes: #263136
Files:
 715a62618707752c54866f253f36af65 554 base required makedev_2.3.1-75.dsc
 a4ec37d2e90f7f04006edca48ec149a9 46382 base required makedev_2.3.1-75.diff.gz
 b3b14f744fe7232112d0ae135830194b 40492 base required makedev_2.3.1-75_all.deb

-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.2.5 (GNU/Linux)

iD8DBQFBI6RmZKfAp/LPAagRAmZJAJ0eRNd+f3KYWBRs+SxJcAbTIPGQmgCdHg2W
WFEhZ+Tbs2iDidHeDAjCKqg=
=5Nxn
-----END PGP SIGNATURE-----

[Debian Bug Importer (debzilla)]

Message-Id: <email address hidden>
Date: Wed, 18 Aug 2004 12:44:12 -0600 (MDT)
From: <email address hidden> (Bdale Garbee)
To: <email address hidden>, <email address hidden>
Subject: patch accepted

tag 263136 +pending
thanks

I've applied the patch in my revision control system for the next upload.

Bdale

[Debian Bug Importer (debzilla)]

Message-Id: <email address hidden>
Date: Wed, 18 Aug 2004 16:03:08 -0400
From: Bdale Garbee <email address hidden>
To: <email address hidden>
Subject: Bug#263136: fixed in makedev 2.3.1-75

Source: makedev
Source-Version: 2.3.1-75

We believe that the bug you reported is fixed in the latest version of
makedev, which is due to be installed in the Debian FTP archive:

makedev_2.3.1-75.diff.gz
  to pool/main/m/makedev/makedev_2.3.1-75.diff.gz
makedev_2.3.1-75.dsc
  to pool/main/m/makedev/makedev_2.3.1-75.dsc
makedev_2.3.1-75_all.deb
  to pool/main/m/makedev/makedev_2.3.1-75_all.deb

A summary of the changes between this version and the previous one is
attached.

Thank you for reporting the bug, which will now be closed. If you
have further comments please address them to <email address hidden>,
and the maintainer will reopen the bug report if appropriate.

Debian distribution maintenance software
pp.
Bdale Garbee <email address hidden> (supplier of updated makedev package)

(This message was generated automatically at their request; if you
believe that there is a problem with it please contact the archive
administrators by mailing <email address hidden>)

-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

Format: 1.7
Date: Wed, 18 Aug 2004 12:42:40 -0600
Source: makedev
Binary: makedev
Architecture: source all
Version: 2.3.1-75
Distribution: unstable
Urgency: low
Maintainer: Bdale Garbee <email address hidden>
Changed-By: Bdale Garbee <email address hidden>
Description:
 makedev - Creates device files in /dev
Closes: 261477 263136
Changes:
 makedev (2.3.1-75) unstable; urgency=low
 .
   * fix one more POSIX compliance issue in postinst, closes: #261477
   * patch from Thomas Schmidt to fix problems in handling DVB devices,
     which change major number at kernel 2.6.8, closes: #263136
Files:
 715a62618707752c54866f253f36af65 554 base required makedev_2.3.1-75.dsc
 a4ec37d2e90f7f04006edca48ec149a9 46382 base required makedev_2.3.1-75.diff.gz
 b3b14f744fe7232112d0ae135830194b 40492 base required makedev_2.3.1-75_all.deb

-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.2.5 (GNU/Linux)

iD8DBQFBI6RmZKfAp/LPAagRAmZJAJ0eRNd+f3KYWBRs+SxJcAbTIPGQmgCdHg2W
WFEhZ+Tbs2iDidHeDAjCKqg=
=5Nxn
-----END PGP SIGNATURE-----

[Thomas Schmidt (tschmidt)]

reopen 263136 <email address hidden>
tags 263136 + sarge

I hope you do not see this as an offending action, but i would really
like to see this issue fixed in sarge.

Regards,
Thomas

--
Thomas Schmidt
:wq

[Debian Bug Importer (debzilla)]

Message-ID: <20040910163021.GA31150@slipstream>
Date: Fri, 10 Sep 2004 18:30:21 +0200
From: Thomas Schmidt <email address hidden>
To: <email address hidden>
Subject: Re: Bug#263136: This issue might break applications

--bg08WKrSYDhXBjb5
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
Content-Transfer-Encoding: quoted-printable

reopen 263136 <email address hidden>
tags 263136 + sarge

I hope you do not see this as an offending action, but i would really
like to see this issue fixed in sarge.

Regards,
Thomas

--=20
Thomas Schmidt
:wq

--bg08WKrSYDhXBjb5
Content-Type: application/pgp-signature; name="signature.asc"
Content-Description: Digital signature
Content-Disposition: inline

-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.2.5 (GNU/Linux)

iD8DBQFBQdadc9+NqwoydlIRApURAJ0flNNPVcJKmXvt6tj9BWKB+ITXOwCgov4N
j2GOMHnkuz/dq0znd6riLGg=
=bjbL
-----END PGP SIGNATURE-----

--bg08WKrSYDhXBjb5--

[Thomas Schmidt (tschmidt)]

The package is now hinted to enter testing, and will enter testing
today, so this bug can be closed again.

--
Thomas Schmidt
:wq

[Debian Bug Importer (debzilla)]

Message-ID: <20040911133621.GA13244@slipstream>
Date: Sat, 11 Sep 2004 15:36:21 +0200
From: Thomas Schmidt <email address hidden>
To: <email address hidden>
Subject: Re: Bug#263136: This issue might break applications

--tThc/1wpZn/ma/RB
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
Content-Transfer-Encoding: quoted-printable

The package is now hinted to enter testing, and will enter testing
today, so this bug can be closed again.

--=20
Thomas Schmidt
:wq

--tThc/1wpZn/ma/RB
Content-Type: application/pgp-signature; name="signature.asc"
Content-Description: Digital signature
Content-Disposition: inline

-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.2.5 (GNU/Linux)

iD8DBQFBQv9Vc9+NqwoydlIRAlouAJ9GqieJxDLN6nhfyzy12G3HHv8RqwCfVAa7
eJWuc3xMMMK8T7wPs30ezbc=
=e4ML
-----END PGP SIGNATURE-----

--tThc/1wpZn/ma/RB--