* [compat-wireless] modprobe -l is legacy compatibility mode and is being removed from some distros
@ 2012-01-24 19:35 Richard Farina
2012-01-29 21:39 ` Hauke Mehrtens
2012-01-31 23:14 ` Luis R. Rodriguez
0 siblings, 2 replies; 8+ messages in thread
From: Richard Farina @ 2012-01-24 19:35 UTC (permalink / raw)
To: Luis R. Rodriguez, Hauke Mehrtens, wireless
module-init-tools has been deprecated in favor of kmod in Arch Linux,
and kmod doesn't have the legacy --list option for modprobe.
This patch fixes the areas of the Makefile which actually needed the
output of modprobe -l. The other 100's of uses are just to echo info to
the screen which in this case will simply fail in an extremely noisy
fashion yet non-fatal way.
Please apply to all branches.
Signed-of-By: Rick Farina <sidhayn@gmail.com>
diff -Naur compat-wireless-3.3-rc1-2/Makefile
compat-wireless-3.3-rc1-2-modinfo-migration/Makefile
--- compat-wireless-3.3-rc1-2/Makefile 2012-01-21 02:00:21.000000000
-0500
+++ compat-wireless-3.3-rc1-2-modinfo-migration/Makefile 2012-01-24
14:27:20.000000000 -0500
@@ -9,11 +9,6 @@
# Sometimes not available in the path
MODPROBE := /sbin/modprobe
-ifneq ($(wildcard $(MODPROBE)),)
-MADWIFI=$(shell $(MODPROBE) -l ath_pci)
-OLD_IWL=$(shell $(MODPROBE) -l iwl4965)
-endif
-
DESTDIR?=
ifneq ($(KERNELRELEASE),)
@@ -188,7 +183,7 @@
@install scripts/athload $(DESTDIR)/usr/sbin/
@install scripts/b43load $(DESTDIR)/usr/sbin/
@install scripts/iwl-load $(DESTDIR)/usr/sbin/
- @if [ ! -z "$(MADWIFI)" ] && [ -z "$(DESTDIR)" ]; then \
+ @if [ `modinfo ath_pci > /dev/null 2>&1` ] && [ -z "$(DESTDIR)" ];
then \
echo ;\
echo -n "Note: madwifi detected, we're going to disable it. " ;\
echo "If you would like to enable it later you can run:" ;\
@@ -197,7 +192,7 @@
echo Running athenable ath5k...;\
/usr/sbin/athenable ath5k ;\
fi
- @if [ ! -z "$(OLD_IWL)" ] && [ -z "$(DESTDIR)" ]; then \
+ @if [ `modinfo iwl4965 > /dev/null 2>&1` ] && [ -z "$(DESTDIR)" ];
then \
echo ;\
echo -n "Note: iwl4965 detected, we're going to disable it. " ;\
echo "If you would like to enable it later you can run:" ;\
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [compat-wireless] modprobe -l is legacy compatibility mode and is being removed from some distros
2012-01-24 19:35 [compat-wireless] modprobe -l is legacy compatibility mode and is being removed from some distros Richard Farina
@ 2012-01-29 21:39 ` Hauke Mehrtens
2012-01-31 14:04 ` Richard Farina
2012-01-31 23:14 ` Luis R. Rodriguez
1 sibling, 1 reply; 8+ messages in thread
From: Hauke Mehrtens @ 2012-01-29 21:39 UTC (permalink / raw)
To: Richard Farina; +Cc: Luis R. Rodriguez, wireless
On 01/24/2012 08:35 PM, Richard Farina wrote:
> module-init-tools has been deprecated in favor of kmod in Arch Linux,
> and kmod doesn't have the legacy --list option for modprobe.
> This patch fixes the areas of the Makefile which actually needed the
> output of modprobe -l. The other 100's of uses are just to echo info to
> the screen which in this case will simply fail in an extremely noisy
> fashion yet non-fatal way.
>
> Please apply to all branches.
>
> Signed-of-By: Rick Farina <sidhayn@gmail.com>
>
> diff -Naur compat-wireless-3.3-rc1-2/Makefile
> compat-wireless-3.3-rc1-2-modinfo-migration/Makefile
> --- compat-wireless-3.3-rc1-2/Makefile 2012-01-21 02:00:21.000000000
> -0500
> +++ compat-wireless-3.3-rc1-2-modinfo-migration/Makefile 2012-01-24
> 14:27:20.000000000 -0500
> @@ -9,11 +9,6 @@
> # Sometimes not available in the path
> MODPROBE := /sbin/modprobe
>
> -ifneq ($(wildcard $(MODPROBE)),)
> -MADWIFI=$(shell $(MODPROBE) -l ath_pci)
> -OLD_IWL=$(shell $(MODPROBE) -l iwl4965)
> -endif
> -
What happens if modinfo is not available? I do not know on what systems
this is the case, but I do not think that modinfo is available when
modprobe is not.
> DESTDIR?=
>
> ifneq ($(KERNELRELEASE),)
> @@ -188,7 +183,7 @@
> @install scripts/athload $(DESTDIR)/usr/sbin/
> @install scripts/b43load $(DESTDIR)/usr/sbin/
> @install scripts/iwl-load $(DESTDIR)/usr/sbin/
> - @if [ ! -z "$(MADWIFI)" ] && [ -z "$(DESTDIR)" ]; then \
> + @if [ `modinfo ath_pci > /dev/null 2>&1` ] && [ -z "$(DESTDIR)" ];
> then \
> echo ;\
> echo -n "Note: madwifi detected, we're going to disable it. " ;\
> echo "If you would like to enable it later you can run:" ;\
> @@ -197,7 +192,7 @@
> echo Running athenable ath5k...;\
> /usr/sbin/athenable ath5k ;\
> fi
> - @if [ ! -z "$(OLD_IWL)" ] && [ -z "$(DESTDIR)" ]; then \
> + @if [ `modinfo iwl4965 > /dev/null 2>&1` ] && [ -z "$(DESTDIR)" ];
> then \
> echo ;\
> echo -n "Note: iwl4965 detected, we're going to disable it. " ;\
> echo "If you would like to enable it later you can run:" ;\
>
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [compat-wireless] modprobe -l is legacy compatibility mode and is being removed from some distros
2012-01-29 21:39 ` Hauke Mehrtens
@ 2012-01-31 14:04 ` Richard Farina
2012-01-31 23:15 ` Luis R. Rodriguez
0 siblings, 1 reply; 8+ messages in thread
From: Richard Farina @ 2012-01-31 14:04 UTC (permalink / raw)
To: Hauke Mehrtens; +Cc: Luis R. Rodriguez, wireless
On 01/29/2012 04:39 PM, Hauke Mehrtens wrote:
> On 01/24/2012 08:35 PM, Richard Farina wrote:
>> module-init-tools has been deprecated in favor of kmod in Arch Linux,
>> and kmod doesn't have the legacy --list option for modprobe.
>> This patch fixes the areas of the Makefile which actually needed the
>> output of modprobe -l. The other 100's of uses are just to echo info to
>> the screen which in this case will simply fail in an extremely noisy
>> fashion yet non-fatal way.
>>
>> Please apply to all branches.
>>
>> Signed-of-By: Rick Farina<sidhayn@gmail.com>
>>
>> diff -Naur compat-wireless-3.3-rc1-2/Makefile
>> compat-wireless-3.3-rc1-2-modinfo-migration/Makefile
>> --- compat-wireless-3.3-rc1-2/Makefile 2012-01-21 02:00:21.000000000
>> -0500
>> +++ compat-wireless-3.3-rc1-2-modinfo-migration/Makefile 2012-01-24
>> 14:27:20.000000000 -0500
>> @@ -9,11 +9,6 @@
>> # Sometimes not available in the path
>> MODPROBE := /sbin/modprobe
>>
>> -ifneq ($(wildcard $(MODPROBE)),)
>> -MADWIFI=$(shell $(MODPROBE) -l ath_pci)
>> -OLD_IWL=$(shell $(MODPROBE) -l iwl4965)
>> -endif
>> -
> What happens if modinfo is not available? I do not know on what systems
> this is the case, but I do not think that modinfo is available when
> modprobe is not.
AFAIK this is never the case. modinfo is packaged with
module-init-tools and on Arch module-init-tools has been replaced by
KMOD which also includes modinfo. It would seem to me that at this
point modinfo is a required tool and is always packaged with modprobe.
-Rick
>> DESTDIR?=
>>
>> ifneq ($(KERNELRELEASE),)
>> @@ -188,7 +183,7 @@
>> @install scripts/athload $(DESTDIR)/usr/sbin/
>> @install scripts/b43load $(DESTDIR)/usr/sbin/
>> @install scripts/iwl-load $(DESTDIR)/usr/sbin/
>> - @if [ ! -z "$(MADWIFI)" ]&& [ -z "$(DESTDIR)" ]; then \
>> + @if [ `modinfo ath_pci> /dev/null 2>&1` ]&& [ -z "$(DESTDIR)" ];
>> then \
>> echo ;\
>> echo -n "Note: madwifi detected, we're going to disable it. " ;\
>> echo "If you would like to enable it later you can run:" ;\
>> @@ -197,7 +192,7 @@
>> echo Running athenable ath5k...;\
>> /usr/sbin/athenable ath5k ;\
>> fi
>> - @if [ ! -z "$(OLD_IWL)" ]&& [ -z "$(DESTDIR)" ]; then \
>> + @if [ `modinfo iwl4965> /dev/null 2>&1` ]&& [ -z "$(DESTDIR)" ];
>> then \
>> echo ;\
>> echo -n "Note: iwl4965 detected, we're going to disable it. " ;\
>> echo "If you would like to enable it later you can run:" ;\
>>
>
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [compat-wireless] modprobe -l is legacy compatibility mode and is being removed from some distros
2012-01-24 19:35 [compat-wireless] modprobe -l is legacy compatibility mode and is being removed from some distros Richard Farina
2012-01-29 21:39 ` Hauke Mehrtens
@ 2012-01-31 23:14 ` Luis R. Rodriguez
2012-02-03 2:50 ` Richard Farina
1 sibling, 1 reply; 8+ messages in thread
From: Luis R. Rodriguez @ 2012-01-31 23:14 UTC (permalink / raw)
To: Richard Farina; +Cc: Hauke Mehrtens, wireless
On Tue, Jan 24, 2012 at 11:35 AM, Richard Farina <sidhayn@gmail.com> wrote:
> module-init-tools has been deprecated in favor of kmod in Arch Linux, and
> kmod doesn't have the legacy --list option for modprobe.
> This patch fixes the areas of the Makefile which actually needed the output
> of modprobe -l. The other 100's of uses are just to echo info to the screen
> which in this case will simply fail in an extremely noisy fashion yet
> non-fatal way.
>
> Please apply to all branches.
>
> Signed-of-By: Rick Farina <sidhayn@gmail.com>
>
> diff -Naur compat-wireless-3.3-rc1-2/Makefile
> compat-wireless-3.3-rc1-2-modinfo-migration/Makefile
> --- compat-wireless-3.3-rc1-2/Makefile 2012-01-21 02:00:21.000000000
> -0500
This patch form is not valid. Not sure if it was your SMTP or what but
its busted and I cannot apply it even if I try to salvage it manually.
Please consider using git on top of the latest master branch:
git://github.com/mcgrof/compat-wireless.git
And then please consider sending me patches using ssmtp or something
that will respect the form so that I can apply the patch accordingly.
For instructions you can read:
http://wireless.kernel.org/en/developers/Documentation/git-guide
You can read the ssmtp section, it covers gmail.
> +++ compat-wireless-3.3-rc1-2-modinfo-migration/Makefile 2012-01-24
> 14:27:20.000000000 -0500
> @@ -9,11 +9,6 @@
> # Sometimes not available in the path
> MODPROBE := /sbin/modprobe
>
> -ifneq ($(wildcard $(MODPROBE)),)
> -MADWIFI=$(shell $(MODPROBE) -l ath_pci)
> -OLD_IWL=$(shell $(MODPROBE) -l iwl4965)
> -endif
> -
> DESTDIR?=
>
> ifneq ($(KERNELRELEASE),)
> @@ -188,7 +183,7 @@
> @install scripts/athload $(DESTDIR)/usr/sbin/
> @install scripts/b43load $(DESTDIR)/usr/sbin/
> @install scripts/iwl-load $(DESTDIR)/usr/sbin/
> - @if [ ! -z "$(MADWIFI)" ] && [ -z "$(DESTDIR)" ]; then \
> + @if [ `modinfo ath_pci > /dev/null 2>&1` ] && [ -z "$(DESTDIR)" ]; then
> \
This check never values to true for me. Did you test it?
$ if [ `modinfo ath9k > /dev/null 2>&1` ] ; then echo module exists;
else echo module does not exist; fi
module does not exist
So at least for me this check does not work. Also consider using
something like $(modinfo ath9k), it looks a lot nicer than the
`modinfo ath9k` stuff.
> echo ;\
> echo -n "Note: madwifi detected, we're going to disable it. " ;\
> echo "If you would like to enable it later you can run:" ;\
> @@ -197,7 +192,7 @@
> echo Running athenable ath5k...;\
> /usr/sbin/athenable ath5k ;\
> fi
> - @if [ ! -z "$(OLD_IWL)" ] && [ -z "$(DESTDIR)" ]; then \
> + @if [ `modinfo iwl4965 > /dev/null 2>&1` ] && [ -z "$(DESTDIR)" ]; then
> \
Same here.
Luis
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [compat-wireless] modprobe -l is legacy compatibility mode and is being removed from some distros
2012-01-31 14:04 ` Richard Farina
@ 2012-01-31 23:15 ` Luis R. Rodriguez
2012-02-03 3:46 ` Rusty Russell
0 siblings, 1 reply; 8+ messages in thread
From: Luis R. Rodriguez @ 2012-01-31 23:15 UTC (permalink / raw)
To: Richard Farina, Rusty Russell; +Cc: Hauke Mehrtens, wireless
On Tue, Jan 31, 2012 at 6:04 AM, Richard Farina <sidhayn@gmail.com> wrote:
> On 01/29/2012 04:39 PM, Hauke Mehrtens wrote:
>>
>> On 01/24/2012 08:35 PM, Richard Farina wrote:
>>>
>>> module-init-tools has been deprecated in favor of kmod in Arch Linux,
>>> and kmod doesn't have the legacy --list option for modprobe.
>>> This patch fixes the areas of the Makefile which actually needed the
>>> output of modprobe -l. The other 100's of uses are just to echo info to
>>> the screen which in this case will simply fail in an extremely noisy
>>> fashion yet non-fatal way.
>>>
>>> Please apply to all branches.
>>>
>>> Signed-of-By: Rick Farina<sidhayn@gmail.com>
>>>
>>> diff -Naur compat-wireless-3.3-rc1-2/Makefile
>>> compat-wireless-3.3-rc1-2-modinfo-migration/Makefile
>>> --- compat-wireless-3.3-rc1-2/Makefile 2012-01-21 02:00:21.000000000
>>> -0500
>>> +++ compat-wireless-3.3-rc1-2-modinfo-migration/Makefile 2012-01-24
>>> 14:27:20.000000000 -0500
>>> @@ -9,11 +9,6 @@
>>> # Sometimes not available in the path
>>> MODPROBE := /sbin/modprobe
>>>
>>> -ifneq ($(wildcard $(MODPROBE)),)
>>> -MADWIFI=$(shell $(MODPROBE) -l ath_pci)
>>> -OLD_IWL=$(shell $(MODPROBE) -l iwl4965)
>>> -endif
>>> -
>>
>> What happens if modinfo is not available? I do not know on what systems
>> this is the case, but I do not think that modinfo is available when
>> modprobe is not.
>
> AFAIK this is never the case. modinfo is packaged with module-init-tools
> and on Arch module-init-tools has been replaced by KMOD which also includes
> modinfo. It would seem to me that at this point modinfo is a required tool
> and is always packaged with modprobe.
Rusty, should all Linux distributions that carry modprobe also have modinfo ?
Luis
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [compat-wireless] modprobe -l is legacy compatibility mode and is being removed from some distros
2012-01-31 23:14 ` Luis R. Rodriguez
@ 2012-02-03 2:50 ` Richard Farina
0 siblings, 0 replies; 8+ messages in thread
From: Richard Farina @ 2012-02-03 2:50 UTC (permalink / raw)
To: Luis R. Rodriguez; +Cc: Hauke Mehrtens, wireless
On 01/31/12 18:14, Luis R. Rodriguez wrote:
> On Tue, Jan 24, 2012 at 11:35 AM, Richard Farina <sidhayn@gmail.com> wrote:
>> module-init-tools has been deprecated in favor of kmod in Arch Linux, and
>> kmod doesn't have the legacy --list option for modprobe.
>> This patch fixes the areas of the Makefile which actually needed the output
>> of modprobe -l. The other 100's of uses are just to echo info to the screen
>> which in this case will simply fail in an extremely noisy fashion yet
>> non-fatal way.
>>
>> Please apply to all branches.
>>
>> Signed-of-By: Rick Farina <sidhayn@gmail.com>
>>
>> diff -Naur compat-wireless-3.3-rc1-2/Makefile
>> compat-wireless-3.3-rc1-2-modinfo-migration/Makefile
>> --- compat-wireless-3.3-rc1-2/Makefile 2012-01-21 02:00:21.000000000
>> -0500
> This patch form is not valid. Not sure if it was your SMTP or what but
> its busted and I cannot apply it even if I try to salvage it manually.
> Please consider using git on top of the latest master branch:
>
> git://github.com/mcgrof/compat-wireless.git
>
> And then please consider sending me patches using ssmtp or something
> that will respect the form so that I can apply the patch accordingly.
> For instructions you can read:
>
> http://wireless.kernel.org/en/developers/Documentation/git-guide
>
> You can read the ssmtp section, it covers gmail.
I read like 10 guides on how to submit patches, I just sent, I hope I
got it right....
>
>> +++ compat-wireless-3.3-rc1-2-modinfo-migration/Makefile 2012-01-24
>> 14:27:20.000000000 -0500
>> @@ -9,11 +9,6 @@
>> # Sometimes not available in the path
>> MODPROBE := /sbin/modprobe
>>
>> -ifneq ($(wildcard $(MODPROBE)),)
>> -MADWIFI=$(shell $(MODPROBE) -l ath_pci)
>> -OLD_IWL=$(shell $(MODPROBE) -l iwl4965)
>> -endif
>> -
>> DESTDIR?=
>>
>> ifneq ($(KERNELRELEASE),)
>> @@ -188,7 +183,7 @@
>> @install scripts/athload $(DESTDIR)/usr/sbin/
>> @install scripts/b43load $(DESTDIR)/usr/sbin/
>> @install scripts/iwl-load $(DESTDIR)/usr/sbin/
>> - @if [ ! -z "$(MADWIFI)" ] && [ -z "$(DESTDIR)" ]; then \
>> + @if [ `modinfo ath_pci > /dev/null 2>&1` ] && [ -z "$(DESTDIR)" ]; then
>> \
> This check never values to true for me. Did you test it?
>
> $ if [ `modinfo ath9k > /dev/null 2>&1` ] ; then echo module exists;
> else echo module does not exist; fi
>
> module does not exist
>
> So at least for me this check does not work. Also consider using
> something like $(modinfo ath9k), it looks a lot nicer than the
> `modinfo ath9k` stuff.
Yeah... I messed up on that check in this patch as well as somewhere
else, the resend fixes it.
Thanks,
Rick Farina
>> echo ;\
>> echo -n "Note: madwifi detected, we're going to disable it. " ;\
>> echo "If you would like to enable it later you can run:" ;\
>> @@ -197,7 +192,7 @@
>> echo Running athenable ath5k...;\
>> /usr/sbin/athenable ath5k ;\
>> fi
>> - @if [ ! -z "$(OLD_IWL)" ] && [ -z "$(DESTDIR)" ]; then \
>> + @if [ `modinfo iwl4965 > /dev/null 2>&1` ] && [ -z "$(DESTDIR)" ]; then
>> \
> Same here.
>
> Luis
>
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [compat-wireless] modprobe -l is legacy compatibility mode and is being removed from some distros
2012-01-31 23:15 ` Luis R. Rodriguez
@ 2012-02-03 3:46 ` Rusty Russell
2012-02-04 13:44 ` Lucas De Marchi
0 siblings, 1 reply; 8+ messages in thread
From: Rusty Russell @ 2012-02-03 3:46 UTC (permalink / raw)
To: Luis R. Rodriguez, Richard Farina
Cc: Hauke Mehrtens, wireless, Lucas De Marchi, Jon Masters
On Tue, 31 Jan 2012 15:15:45 -0800, "Luis R. Rodriguez" <mcgrof@gmail.com> wrote:
> On Tue, Jan 31, 2012 at 6:04 AM, Richard Farina <sidhayn@gmail.com> wrote:
> > On 01/29/2012 04:39 PM, Hauke Mehrtens wrote:
> >>
> >> On 01/24/2012 08:35 PM, Richard Farina wrote:
> >>>
> >>> module-init-tools has been deprecated in favor of kmod in Arch Linux,
> >>> and kmod doesn't have the legacy --list option for modprobe.
> >>> This patch fixes the areas of the Makefile which actually needed the
> >>> output of modprobe -l. The other 100's of uses are just to echo info to
> >>> the screen which in this case will simply fail in an extremely noisy
> >>> fashion yet non-fatal way.
> >>>
> >>> Please apply to all branches.
> >>>
> >>> Signed-of-By: Rick Farina<sidhayn@gmail.com>
> >>>
> >>> diff -Naur compat-wireless-3.3-rc1-2/Makefile
> >>> compat-wireless-3.3-rc1-2-modinfo-migration/Makefile
> >>> --- compat-wireless-3.3-rc1-2/Makefile 2012-01-21 02:00:21.000000000
> >>> -0500
> >>> +++ compat-wireless-3.3-rc1-2-modinfo-migration/Makefile 2012-01-24
> >>> 14:27:20.000000000 -0500
> >>> @@ -9,11 +9,6 @@
> >>> # Sometimes not available in the path
> >>> MODPROBE := /sbin/modprobe
> >>>
> >>> -ifneq ($(wildcard $(MODPROBE)),)
> >>> -MADWIFI=$(shell $(MODPROBE) -l ath_pci)
> >>> -OLD_IWL=$(shell $(MODPROBE) -l iwl4965)
> >>> -endif
> >>> -
> >>
> >> What happens if modinfo is not available? I do not know on what systems
> >> this is the case, but I do not think that modinfo is available when
> >> modprobe is not.
> >
> > AFAIK this is never the case. modinfo is packaged with module-init-tools
> > and on Arch module-init-tools has been replaced by KMOD which also includes
> > modinfo. It would seem to me that at this point modinfo is a required tool
> > and is always packaged with modprobe.
>
> Rusty, should all Linux distributions that carry modprobe also have modinfo ?
Yes, at least so far. But you should make sure the kmod guys are in the
loop, so they know exactly what you're trying to do.
This feedback allows them to decide whether to offer it in a different
way...
Cheers,
Rusty.
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [compat-wireless] modprobe -l is legacy compatibility mode and is being removed from some distros
2012-02-03 3:46 ` Rusty Russell
@ 2012-02-04 13:44 ` Lucas De Marchi
0 siblings, 0 replies; 8+ messages in thread
From: Lucas De Marchi @ 2012-02-04 13:44 UTC (permalink / raw)
To: Rusty Russell
Cc: Luis R. Rodriguez, Richard Farina, Hauke Mehrtens, wireless,
Jon Masters, linux-modules
* Rusty Russell <rusty@rustcorp.com.au> [2012-02-03 14:16:30 +1030]:
> On Tue, 31 Jan 2012 15:15:45 -0800, "Luis R. Rodriguez" <mcgrof@gmail.com> wrote:
> > On Tue, Jan 31, 2012 at 6:04 AM, Richard Farina <sidhayn@gmail.com> wrote:
> > > On 01/29/2012 04:39 PM, Hauke Mehrtens wrote:
> > >>
> > >> On 01/24/2012 08:35 PM, Richard Farina wrote:
> > >>>
> > >>> module-init-tools has been deprecated in favor of kmod in Arch Linux,
> > >>> and kmod doesn't have the legacy --list option for modprobe.
> > >>> This patch fixes the areas of the Makefile which actually needed the
> > >>> output of modprobe -l. The other 100's of uses are just to echo info to
> > >>> the screen which in this case will simply fail in an extremely noisy
> > >>> fashion yet non-fatal way.
> > >>>
> > >>> Please apply to all branches.
> > >>>
> > >>> Signed-of-By: Rick Farina<sidhayn@gmail.com>
> > >>>
> > >>> diff -Naur compat-wireless-3.3-rc1-2/Makefile
> > >>> compat-wireless-3.3-rc1-2-modinfo-migration/Makefile
> > >>> --- compat-wireless-3.3-rc1-2/Makefile 2012-01-21 02:00:21.000000000
> > >>> -0500
> > >>> +++ compat-wireless-3.3-rc1-2-modinfo-migration/Makefile 2012-01-24
> > >>> 14:27:20.000000000 -0500
> > >>> @@ -9,11 +9,6 @@
> > >>> # Sometimes not available in the path
> > >>> MODPROBE := /sbin/modprobe
> > >>>
> > >>> -ifneq ($(wildcard $(MODPROBE)),)
> > >>> -MADWIFI=$(shell $(MODPROBE) -l ath_pci)
> > >>> -OLD_IWL=$(shell $(MODPROBE) -l iwl4965)
> > >>> -endif
> > >>> -
Why do you need to know this? If you really need this, it can be well
accomplished by 'find'. That's the reason it was removed from modprobe
(and it was already deprecated in module-init-tools).
But what's the point in knowing where the file is?
> > >>
> > >> What happens if modinfo is not available? I do not know on what systems
> > >> this is the case, but I do not think that modinfo is available when
> > >> modprobe is not.
> > >
> > > AFAIK this is never the case. modinfo is packaged with module-init-tools
> > > and on Arch module-init-tools has been replaced by KMOD which also includes
> > > modinfo. It would seem to me that at this point modinfo is a required tool
> > > and is always packaged with modprobe.
> >
> > Rusty, should all Linux distributions that carry modprobe also have modinfo ?
>
> Yes, at least so far. But you should make sure the kmod guys are in the
> loop, so they know exactly what you're trying to do.
>
Yes, let us know what exactly you are trying to accomplish here.
Lucas De Marchi
^ permalink raw reply [flat|nested] 8+ messages in thread
end of thread, other threads:[~2012-02-04 13:42 UTC | newest]
Thread overview: 8+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2012-01-24 19:35 [compat-wireless] modprobe -l is legacy compatibility mode and is being removed from some distros Richard Farina
2012-01-29 21:39 ` Hauke Mehrtens
2012-01-31 14:04 ` Richard Farina
2012-01-31 23:15 ` Luis R. Rodriguez
2012-02-03 3:46 ` Rusty Russell
2012-02-04 13:44 ` Lucas De Marchi
2012-01-31 23:14 ` Luis R. Rodriguez
2012-02-03 2:50 ` Richard Farina
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).