Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 596480 - >=net-misc/grive-0.5.1_pre20160706 should depend on net-misc/curl[curl_ssl_gnutls]
Summary: >=net-misc/grive-0.5.1_pre20160706 should depend on net-misc/curl[curl_ssl_gn...
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: All Linux
: Normal normal (vote)
Assignee: Christoph Junghans (RETIRED)
URL:
Whiteboard:
Keywords: PATCH
Depends on:
Blocks:
 
Reported: 2016-10-07 19:30 UTC by INODE64 Sistemas
Modified: 2016-10-19 11:20 UTC (History)
1 user (show)

See Also:
Package list:
Runtime testing required: ---

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description INODE64 Sistemas 2016-10-07 19:30:51 UTC 
See the commit for new dependences

https://github.com/vitalif/grive2/commit/457d84974592ed3f123622b12f2a7a748cbc0668


Reproducible: Always
Comment 1 INODE64 Sistemas 2016-10-07 19:32:13 UTC 
The patch

--- grive-0.5.1_pre20160114.ebuild  2016-02-06 23:46:37.000000000 +0100
+++ grive-9999.ebuild   2016-10-07 21:14:44.506644260 +0200
@@ -28,7 +28,7 @@
        dev-libs/expat
        >=dev-libs/json-c-0.11-r1:=
        dev-libs/libgcrypt:0=
-       net-misc/curl
+       net-misc/curl[curl_ssl_gnutls]
        sys-libs/binutils-libs:0=
        sys-libs/glibc
        dev-libs/yajl
Comment 2 Christoph Junghans (RETIRED) gentoo-dev 2016-10-17 08:32:01 UTC 
net-misc/grive-0.5.1_pre20160706 wasn't actually in the tree, so I did a version bump and included that fix.

commit 71348e79553ae2831f12aa787b48b54c2645428c
Author: Christoph Junghans <ottxor@gentoo.org>
Date:   Mon Oct 17 02:29:39 2016 -0600

    net-misc/grive: version bump (fix bug #596480)

    Package-Manager: portage-2.3.0
Comment 3 eroen 2016-10-17 17:37:38 UTC 
(In reply to INODE64 Sistemas from comment #1)
> The patch
> 
> --- grive-0.5.1_pre20160114.ebuild  2016-02-06 23:46:37.000000000 +0100
> +++ grive-9999.ebuild   2016-10-07 21:14:44.506644260 +0200
> @@ -28,7 +28,7 @@
>         dev-libs/expat
>         >=dev-libs/json-c-0.11-r1:=
>         dev-libs/libgcrypt:0=
> -       net-misc/curl
> +       net-misc/curl[curl_ssl_gnutls]
>         sys-libs/binutils-libs:0=
>         sys-libs/glibc
>         dev-libs/yajl

I am not intimately familiar with debian's packaging system, but the upstream change (from [1]) is:

- ... zlib1g-dev, libcurl4-openssl-dev, libstdc++-6-dev | ...
+ ... zlib1g-dev, libcurl4-openssl-dev | libcurl4-gnutls-dev, libstdc++-6-dev | ...

I would assume this means curl-gnutls is now an *option* to curl-openssl, but both are still permissible.

ottxor: Please reopen the bug or respond with a comment, I don't have editbugs and I don't see you in irc :P

1: https://github.com/vitalif/grive2/commit/457d84974592ed3f123622b12f2a7a748cbc066
Comment 4 Christoph Junghans (RETIRED) gentoo-dev 2016-10-17 23:34:29 UTC 
Re-opened, just send me a pull-request on github for an -r1 of net-misc/grive-0.5.1_pre20160706.
Comment 5 Christoph Junghans (RETIRED) gentoo-dev 2016-10-19 11:20:47 UTC 
commit 635d138957cdaa1ec89ec632402afb9b51335db9
Author: Christoph Junghans <ottxor@gentoo.org>
Date:   Wed Oct 19 05:18:51 2016 -0600

    net-misc/grive: fix curl dep

    Package-Manager: portage-2.3.0