Patchwork Change library from libiscsi to libiscsiclient to match library rename in libiscsi

login
register
mail settings
Submitter ronniesahlberg@gmail.com
Date March 3, 2012, 6:38 a.m.
Message ID <1330756709-12781-2-git-send-email-ronniesahlberg@gmail.com>
Download mbox | patch
Permalink /patch/144407/
State New
Headers show

Comments

ronniesahlberg@gmail.com - March 3, 2012, 6:38 a.m.
Signed-off-by: Ronnie Sahlberg <ronniesahlberg@gmail.com>
---
 configure |    4 ++--
 1 files changed, 2 insertions(+), 2 deletions(-)
Michael Tokarev - March 3, 2012, 9:40 a.m.
On 03.03.2012 10:38, Ronnie Sahlberg wrote:
> Signed-off-by: Ronnie Sahlberg <ronniesahlberg@gmail.com>
> ---
>  configure |    4 ++--
>  1 files changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/configure b/configure
> index fb0e18e..294c0c1 100755
> --- a/configure
> +++ b/configure
> @@ -2503,9 +2503,9 @@ if test "$libiscsi" != "no" ; then
>  #include <iscsi/iscsi.h>
>  int main(void) { iscsi_create_context(""); return 0; }
>  EOF
> -  if compile_prog "-Werror" "-liscsi" ; then
> +  if compile_prog "-Werror" "-liscsiclient" ; then
>      libiscsi="yes"
> -    LIBS="$LIBS -liscsi"
> +    LIBS="$LIBS -liscsiclient"
>    else
>      if test "$libiscsi" = "yes" ; then
>        feature_not_found "libiscsi"

Shouldn't we rename the feature and the --with/--without
name too here?  This appears to be partial "fix" for the
name change.

For the feature, I'd suggest using --with-iscsi instead of
--with-libiscsiclient.  But... dunno.

Thanks,

/mjt
ronniesahlberg@gmail.com - March 7, 2012, 5:37 a.m.
Please ignore this patch. We do not need it.



On Sat, Mar 3, 2012 at 8:40 PM, Michael Tokarev <mjt@tls.msk.ru> wrote:
> On 03.03.2012 10:38, Ronnie Sahlberg wrote:
>> Signed-off-by: Ronnie Sahlberg <ronniesahlberg@gmail.com>
>> ---
>>  configure |    4 ++--
>>  1 files changed, 2 insertions(+), 2 deletions(-)
>>
>> diff --git a/configure b/configure
>> index fb0e18e..294c0c1 100755
>> --- a/configure
>> +++ b/configure
>> @@ -2503,9 +2503,9 @@ if test "$libiscsi" != "no" ; then
>>  #include <iscsi/iscsi.h>
>>  int main(void) { iscsi_create_context(""); return 0; }
>>  EOF
>> -  if compile_prog "-Werror" "-liscsi" ; then
>> +  if compile_prog "-Werror" "-liscsiclient" ; then
>>      libiscsi="yes"
>> -    LIBS="$LIBS -liscsi"
>> +    LIBS="$LIBS -liscsiclient"
>>    else
>>      if test "$libiscsi" = "yes" ; then
>>        feature_not_found "libiscsi"
>
> Shouldn't we rename the feature and the --with/--without
> name too here?  This appears to be partial "fix" for the
> name change.
>
> For the feature, I'd suggest using --with-iscsi instead of
> --with-libiscsiclient.  But... dunno.
>
> Thanks,
>
> /mjt

Patch

diff --git a/configure b/configure
index fb0e18e..294c0c1 100755
--- a/configure
+++ b/configure
@@ -2503,9 +2503,9 @@  if test "$libiscsi" != "no" ; then
 #include <iscsi/iscsi.h>
 int main(void) { iscsi_create_context(""); return 0; }
 EOF
-  if compile_prog "-Werror" "-liscsi" ; then
+  if compile_prog "-Werror" "-liscsiclient" ; then
     libiscsi="yes"
-    LIBS="$LIBS -liscsi"
+    LIBS="$LIBS -liscsiclient"
   else
     if test "$libiscsi" = "yes" ; then
       feature_not_found "libiscsi"