Github messages for voidlinux
 help / color / mirror / Atom feed
* [PR PATCH] curl: move license to libcurl subpackage.
@ 2022-06-25  1:07 ericonr
  2022-06-25 13:28 ` sgn
                   ` (2 more replies)
  0 siblings, 3 replies; 4+ messages in thread
From: ericonr @ 2022-06-25  1:07 UTC (permalink / raw)
  To: ml

[-- Attachment #1: Type: text/plain, Size: 1402 bytes --]

There is a new pull request by ericonr against master on the void-packages repository

https://github.com/ericonr/void-packages curl
https://github.com/void-linux/void-packages/pull/37674

curl: move license to libcurl subpackage.
Fixes: #33781

Bikeshed: if we do this for libcurl, I feel like it should be done for all library + tools combo, which doesn't like a fun time or particularly escalable. Thoughts?

<!-- Uncomment relevant sections and delete options which are not applicable -->

#### Testing the changes
- I tested the changes in this PR: **YES**

<!--
#### New package
- This new package conforms to the [quality requirements](https://github.com/void-linux/void-packages/blob/master/Manual.md#quality-requirements): **YES**|**NO**
-->

<!-- Note: If the build is likely to take more than 2 hours, please add ci skip tag as described in
https://github.com/void-linux/void-packages/blob/master/CONTRIBUTING.md#continuous-integration
and test at least one native build and, if supported, at least one cross build.
Ignore this section if this PR is not skipping CI.
-->
<!--
#### Local build testing
- I built this PR locally for my native architecture, (ARCH-LIBC)
- I built this PR locally for these architectures (if supported. mark crossbuilds):
  - aarch64-musl
  - armv7l
  - armv6l-musl
-->


A patch file from https://github.com/void-linux/void-packages/pull/37674.patch is attached

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: github-pr-curl-37674.patch --]
[-- Type: text/x-diff, Size: 1092 bytes --]

From 7390f979f40c0f37ed5b23aa9cfd13705a517901 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?=C3=89rico=20Nogueira?= <erico.erc@gmail.com>
Date: Fri, 24 Jun 2022 22:05:16 -0300
Subject: [PATCH] curl: move license to libcurl subpackage.

Fixes: #33781
---
 srcpkgs/curl/template | 5 ++---
 1 file changed, 2 insertions(+), 3 deletions(-)

diff --git a/srcpkgs/curl/template b/srcpkgs/curl/template
index 00cbfd709875..4aef867304ec 100644
--- a/srcpkgs/curl/template
+++ b/srcpkgs/curl/template
@@ -1,7 +1,7 @@
 # Template file for 'curl'
 pkgname=curl
 version=7.83.1
-revision=1
+revision=2
 build_style=gnu-configure
 configure_args="ac_cv_sizeof_off_t=8 --enable-threaded-resolver --enable-ipv6
  $(vopt_with rtmp) $(vopt_with gssapi) $(vopt_enable ldap) $(vopt_with gnutls)
@@ -60,14 +60,13 @@ post_install() {
 		vsed -i $DESTDIR/usr/bin/curl-config \
 			-e "/[	 ]*--static-libs)/,/[	 ]*;;/ s,-L$XBPS_CROSS_BASE,-L,"
 	fi
-
-	vlicense COPYING
 }
 
 libcurl_package() {
 	short_desc="Multiprotocol file transfer library"
 	pkg_install() {
 		vmove "usr/lib/*.so.*"
+		vlicense COPYING
 	}
 }
 

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: curl: move license to libcurl subpackage.
  2022-06-25  1:07 [PR PATCH] curl: move license to libcurl subpackage ericonr
@ 2022-06-25 13:28 ` sgn
  2022-07-17 19:29 ` [PR PATCH] [Closed]: " ericonr
  2022-07-17 19:29 ` ericonr
  2 siblings, 0 replies; 4+ messages in thread
From: sgn @ 2022-06-25 13:28 UTC (permalink / raw)
  To: ml

[-- Attachment #1: Type: text/plain, Size: 187 bytes --]

New comment by sgn on void-packages repository

https://github.com/void-linux/void-packages/pull/37674#issuecomment-1166286378

Comment:
I think we can move it when updating, one by one.

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: curl: move license to libcurl subpackage.
  2022-06-25  1:07 [PR PATCH] curl: move license to libcurl subpackage ericonr
  2022-06-25 13:28 ` sgn
  2022-07-17 19:29 ` [PR PATCH] [Closed]: " ericonr
@ 2022-07-17 19:29 ` ericonr
  2 siblings, 0 replies; 4+ messages in thread
From: ericonr @ 2022-07-17 19:29 UTC (permalink / raw)
  To: ml

[-- Attachment #1: Type: text/plain, Size: 183 bytes --]

New comment by ericonr on void-packages repository

https://github.com/void-linux/void-packages/pull/37674#issuecomment-1186593984

Comment:
Will be pushing this separately in a bit.

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [PR PATCH] [Closed]: curl: move license to libcurl subpackage.
  2022-06-25  1:07 [PR PATCH] curl: move license to libcurl subpackage ericonr
  2022-06-25 13:28 ` sgn
@ 2022-07-17 19:29 ` ericonr
  2022-07-17 19:29 ` ericonr
  2 siblings, 0 replies; 4+ messages in thread
From: ericonr @ 2022-07-17 19:29 UTC (permalink / raw)
  To: ml

[-- Attachment #1: Type: text/plain, Size: 1254 bytes --]

There's a closed pull request on the void-packages repository

curl: move license to libcurl subpackage.
https://github.com/void-linux/void-packages/pull/37674

Description:
Fixes: #33781

Bikeshed: if we do this for libcurl, I feel like it should be done for all library + tools combo, which doesn't like a fun time or particularly escalable. Thoughts?

<!-- Uncomment relevant sections and delete options which are not applicable -->

#### Testing the changes
- I tested the changes in this PR: **YES**

<!--
#### New package
- This new package conforms to the [quality requirements](https://github.com/void-linux/void-packages/blob/master/Manual.md#quality-requirements): **YES**|**NO**
-->

<!-- Note: If the build is likely to take more than 2 hours, please add ci skip tag as described in
https://github.com/void-linux/void-packages/blob/master/CONTRIBUTING.md#continuous-integration
and test at least one native build and, if supported, at least one cross build.
Ignore this section if this PR is not skipping CI.
-->
<!--
#### Local build testing
- I built this PR locally for my native architecture, (ARCH-LIBC)
- I built this PR locally for these architectures (if supported. mark crossbuilds):
  - aarch64-musl
  - armv7l
  - armv6l-musl
-->


^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2022-07-17 19:29 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-06-25  1:07 [PR PATCH] curl: move license to libcurl subpackage ericonr
2022-06-25 13:28 ` sgn
2022-07-17 19:29 ` [PR PATCH] [Closed]: " ericonr
2022-07-17 19:29 ` ericonr

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).