* [PATCH] staging: hv: make HYPERV_UTILS depend on NLS to prevent a build error
@ 2011-04-25 23:52 Peter Foley
2011-04-26 0:00 ` Greg KH
2011-05-03 17:57 ` Greg KH
0 siblings, 2 replies; 6+ messages in thread
From: Peter Foley @ 2011-04-25 23:52 UTC (permalink / raw)
To: linux-kernel; +Cc: gregkh, hjanssen, haiyangz
This patch adds a dependency on NLS to HYPERV_UTILS to avoid a build error.
Signed-off-by: Peter Foley <pefoley2@verizon.net>
---
drivers/staging/hv/Kconfig | 2 +-
1 files changed, 1 insertions(+), 1 deletions(-)
diff --git a/drivers/staging/hv/Kconfig b/drivers/staging/hv/Kconfig
index d41f380..76f0756 100644
--- a/drivers/staging/hv/Kconfig
+++ b/drivers/staging/hv/Kconfig
@@ -31,7 +31,7 @@ config HYPERV_NET
config HYPERV_UTILS
tristate "Microsoft Hyper-V Utilities driver"
- depends on CONNECTOR
+ depends on CONNECTOR && NLS
default HYPERV
help
Select this option to enable the Hyper-V Utilities.
--
1.7.5.rc1
^ permalink raw reply related [flat|nested] 6+ messages in thread
* Re: [PATCH] staging: hv: make HYPERV_UTILS depend on NLS to prevent a build error
2011-04-25 23:52 [PATCH] staging: hv: make HYPERV_UTILS depend on NLS to prevent a build error Peter Foley
@ 2011-04-26 0:00 ` Greg KH
2011-04-26 0:53 ` Peter Foley
2011-05-03 17:57 ` Greg KH
1 sibling, 1 reply; 6+ messages in thread
From: Greg KH @ 2011-04-26 0:00 UTC (permalink / raw)
To: Peter Foley; +Cc: linux-kernel, hjanssen, haiyangz
On Mon, Apr 25, 2011 at 07:52:18PM -0400, Peter Foley wrote:
> This patch adds a dependency on NLS to HYPERV_UTILS to avoid a build error.
What is the build error?
thanks,
greg k-h
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] staging: hv: make HYPERV_UTILS depend on NLS to prevent a build error
2011-04-26 0:00 ` Greg KH
@ 2011-04-26 0:53 ` Peter Foley
2011-05-03 17:33 ` Greg KH
0 siblings, 1 reply; 6+ messages in thread
From: Peter Foley @ 2011-04-26 0:53 UTC (permalink / raw)
To: Greg KH; +Cc: linux-kernel, hjanssen, haiyangz
On 4/25/2011 8:00 PM, Greg KH wrote:
> On Mon, Apr 25, 2011 at 07:52:18PM -0400, Peter Foley wrote:
>> This patch adds a dependency on NLS to HYPERV_UTILS to avoid a build error.
>
> What is the build error?
>
> thanks,
>
> greg k-h
When CONFIG_HYPERV_UTILS=y and CONFIG_NLS=n
the build fails with this error:
Building modules, stage 2.
MODPOST 3 modules
ERROR: "utf8s_to_utf16s" [drivers/staging/hv/hv_utils.ko] undefined!
make[3]: *** [__modpost] Error 1
make[2]: *** [modules] Error 2
make[2]: *** Waiting for unfinished jobs....
Kernel: arch/x86/boot/bzImage is ready (#2)
make[1]: *** [sub-make] Error 2
make: *** [all] Error 2
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] staging: hv: make HYPERV_UTILS depend on NLS to prevent a build error
2011-04-26 0:53 ` Peter Foley
@ 2011-05-03 17:33 ` Greg KH
0 siblings, 0 replies; 6+ messages in thread
From: Greg KH @ 2011-05-03 17:33 UTC (permalink / raw)
To: Peter Foley; +Cc: Greg KH, linux-kernel, hjanssen, haiyangz
On Mon, Apr 25, 2011 at 08:53:26PM -0400, Peter Foley wrote:
> On 4/25/2011 8:00 PM, Greg KH wrote:
> > On Mon, Apr 25, 2011 at 07:52:18PM -0400, Peter Foley wrote:
> >> This patch adds a dependency on NLS to HYPERV_UTILS to avoid a build error.
> >
> > What is the build error?
> >
> > thanks,
> >
> > greg k-h
>
>
> When CONFIG_HYPERV_UTILS=y and CONFIG_NLS=n
> the build fails with this error:
>
> Building modules, stage 2.
> MODPOST 3 modules
> ERROR: "utf8s_to_utf16s" [drivers/staging/hv/hv_utils.ko] undefined!
> make[3]: *** [__modpost] Error 1
> make[2]: *** [modules] Error 2
> make[2]: *** Waiting for unfinished jobs....
> Kernel: arch/x86/boot/bzImage is ready (#2)
> make[1]: *** [sub-make] Error 2
> make: *** [all] Error 2
Ah, ok, we need this for that function, that's what I wanted to know,
I'll put it in the changelog entry. Please do that the next time.
thanks,
greg k-h
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] staging: hv: make HYPERV_UTILS depend on NLS to prevent a build error
2011-04-25 23:52 [PATCH] staging: hv: make HYPERV_UTILS depend on NLS to prevent a build error Peter Foley
2011-04-26 0:00 ` Greg KH
@ 2011-05-03 17:57 ` Greg KH
2011-05-03 18:11 ` Peter Foley
1 sibling, 1 reply; 6+ messages in thread
From: Greg KH @ 2011-05-03 17:57 UTC (permalink / raw)
To: Peter Foley; +Cc: linux-kernel, gregkh, hjanssen, haiyangz
On Mon, Apr 25, 2011 at 07:52:18PM -0400, Peter Foley wrote:
> This patch adds a dependency on NLS to HYPERV_UTILS to avoid a build error.
>
> Signed-off-by: Peter Foley <pefoley2@verizon.net>
> ---
> drivers/staging/hv/Kconfig | 2 +-
> 1 files changed, 1 insertions(+), 1 deletions(-)
>
> diff --git a/drivers/staging/hv/Kconfig b/drivers/staging/hv/Kconfig
> index d41f380..76f0756 100644
> --- a/drivers/staging/hv/Kconfig
> +++ b/drivers/staging/hv/Kconfig
> @@ -31,7 +31,7 @@ config HYPERV_NET
> config HYPERV_UTILS
> tristate "Microsoft Hyper-V Utilities driver"
> - depends on CONNECTOR
> + depends on CONNECTOR && NLS
This patch doesn't apply at all, and I can't figure out why. Did your
email client somehow mess it up?
Care to resend it, with the proper information in the Changelog entry
(i.e. what build error this is fixing) so I can apply it?
thanks,
greg k-h
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] staging: hv: make HYPERV_UTILS depend on NLS to prevent a build error
2011-05-03 17:57 ` Greg KH
@ 2011-05-03 18:11 ` Peter Foley
0 siblings, 0 replies; 6+ messages in thread
From: Peter Foley @ 2011-05-03 18:11 UTC (permalink / raw)
To: Greg KH; +Cc: linux-kernel, hjanssen, haiyangz
On 5/3/2011 1:57 PM, Greg KH wrote:
> On Mon, Apr 25, 2011 at 07:52:18PM -0400, Peter Foley wrote:
>> This patch adds a dependency on NLS to HYPERV_UTILS to avoid a build error.
>>
>> Signed-off-by: Peter Foley <pefoley2@verizon.net>
>> ---
>> drivers/staging/hv/Kconfig | 2 +-
>> 1 files changed, 1 insertions(+), 1 deletions(-)
>>
>> diff --git a/drivers/staging/hv/Kconfig b/drivers/staging/hv/Kconfig
>> index d41f380..76f0756 100644
>> --- a/drivers/staging/hv/Kconfig
>> +++ b/drivers/staging/hv/Kconfig
>> @@ -31,7 +31,7 @@ config HYPERV_NET
>> config HYPERV_UTILS
>> tristate "Microsoft Hyper-V Utilities driver"
>> - depends on CONNECTOR
>> + depends on CONNECTOR && NLS
>
> This patch doesn't apply at all, and I can't figure out why. Did your
> email client somehow mess it up?
>
> Care to resend it, with the proper information in the Changelog entry
> (i.e. what build error this is fixing) so I can apply it?
>
> thanks,
>
> greg k-h
Ok, I'll resend it.
I seem to have a problem with my email client corrupting patches.
I'm going to try a different one which will hopefully fix the problem.
Thanks,
Peter
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2011-05-03 18:12 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2011-04-25 23:52 [PATCH] staging: hv: make HYPERV_UTILS depend on NLS to prevent a build error Peter Foley
2011-04-26 0:00 ` Greg KH
2011-04-26 0:53 ` Peter Foley
2011-05-03 17:33 ` Greg KH
2011-05-03 17:57 ` Greg KH
2011-05-03 18:11 ` Peter Foley
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox