Bug 676106 – sci-geosciences/viking-1.7 - configure: error: oauth.h is needed but not found - you will need to install package 'liboauth-dev' or similar. The feature can be disabled with --disable-oauth

• Status: RESOLVED FIXED
• Product: Gentoo Linux

[Description Ed Santiago 2019-01-23 13:12:58 UTC]

Created attachment 562574 [details, diff]
Patch for viking-1.7.ebuild, adding optional USE=oauth

Package fails to configure:

checking whether to enable OAuth Support... (cached) yes                       
checking oauth.h usability... no                                               
checking oauth.h presence... no                                                
checking for oauth.h... no
configure: error: oauth.h is needed but not found - you will need to install package 'liboauth-dev' or similar. The feature can be disabled with --disable-oauth

Attached diff adds a USE dependency on oauth; I tested the negative case but not the positive one (USE=oauth enabled). If I read #673572 correctly it sounds like a simpler solution would be to just add --disable-oauth to the ebuild. I'll defer to the maintainer as to which is the best option.

[Comment 1 Larry the Git Cow 2019-02-01 17:01:41 UTC]

The bug has been closed via the following commit(s):

https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=6e21549fbec082e230dcd11b7e014470f376293f

commit 6e21549fbec082e230dcd11b7e014470f376293f
Author:     Andreas K. Hüttel <[e-mail]>
AuthorDate: 2019-02-01 17:01:01 +0000
Commit:     Andreas K. Hüttel <[e-mail]>
CommitDate: 2019-02-01 17:01:26 +0000

    sci-geosciences/viking: Add missing dependency liboauth
    
    Closes: https://bugs.gentoo.org/676106
    Closes: https://bugs.gentoo.org/676132
    Package-Manager: Portage-2.3.59, Repoman-2.3.12
    Signed-off-by: Andreas K. Hüttel <[e-mail]>

 sci-geosciences/viking/viking-1.7.ebuild | 1 +
 1 file changed, 1 insertion(+)

[Comment 2 Ed Santiago 2019-02-02 14:06:57 UTC]

The new ebuild adds net-libs/liboauth as a (mandatory) COMMONDEPEND; this in turn causes my emerge to fail with:

    The following USE changes are necessary to proceed:
 (see "package.use" in the portage(5) man page for more details)
    # required by net-libs/liboauth-1.0.3::gentoo[curl,nss]
    # required by sci-geosciences/viking-1.7::gentoo
    # required by @selected
    # required by @world (argument)
    >=net-misc/curl-7.62.0 curl_ssl_nss

    !!! The ebuild selected to satisfy "net-misc/curl[ssl,curl_ssl_nss]" has unmet requirements.
    - net-misc/curl-7.62.0::gentoo USE="ipv6 ssl -adns -brotli -http2 -idn -kerberos -ldap -metalink -rtmp -samba -ssh -static-libs -test -threads" ABI_X86="(64) -32 (-x32)" CURL_SSL="openssl -axtls -gnutls -libressl -mbedtls -nss (-winssl)"

      The following REQUIRED_USE flag constraints are unsatisfied:
        ssl? ( exactly-one-of ( curl_ssl_axtls curl_ssl_gnutls curl_ssl_libressl curl_ssl_mbedtls curl_ssl_nss curl_ssl_openssl curl_ssl_winssl ) )

      The above constraints are a subset of the following complete expression:
        curl_ssl_winssl? ( elibc_Winnt ) threads? ( !adns ) ssl? ( exactly-one-of ( curl_ssl_axtls curl_ssl_gnutls curl_ssl_libressl curl_ssl_mbedtls curl_ssl_nss curl_ssl_openssl curl_ssl_winssl ) )

Adding curl_ssl_nss to curl is leading me down a dependency rathole. Would you consider making oauth conditional on USE=oauth, as in the patch I submitted?

[Comment 3 Martin Dummer 2019-02-03 10:13:57 UTC]

I can confirm that the automatic indirect addition of net-misc/curl is problematic. I solved it for me with "USE=-curl" for net-libs/liboauth.

But if liboauth is not mandatory for basic functionality of sci-geosciences/viking, I would definitely prefer adding a useflag "oauth" to the viking ebuild!!!

[Comment 4 Martin Dummer 2019-02-03 10:17:12 UTC]

[e-mail] - can you please re-think about just adding the dependency in bug #676132 ?!?

[Comment 5 Larry the Git Cow 2019-02-03 10:49:08 UTC]

The bug has been closed via the following commit(s):

https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=0e131ed5dccd5a31b4fea82c3046eaa522667b50

commit 0e131ed5dccd5a31b4fea82c3046eaa522667b50
Author:     Hans de Graaff <[e-mail]>
AuthorDate: 2019-02-03 07:03:07 +0000
Commit:     Hans de Graaff <[e-mail]>
CommitDate: 2019-02-03 10:03:48 +0000

    sci-geosciences/viking: make oauth support optional
    
    Add USE flag for oauth support based on patch by Ed Santiago.
    
    Fixes: https://bugs.gentoo.org/676106
    Signed-off-by: Hans de Graaff <[e-mail]>
    Package-Manager: Portage-2.3.51, Repoman-2.3.11

 sci-geosciences/viking/metadata.xml      | 3 +++
 sci-geosciences/viking/viking-1.7.ebuild | 5 +++--
 2 files changed, 6 insertions(+), 2 deletions(-)