Github messages for voidlinux
 help / color / mirror / Atom feed
* [PR PATCH] cyrus-sasl-modules: remove dependency to main package
@ 2021-03-10 18:37 ailiop-git
  2021-03-11 20:58 ` ericonr
                   ` (4 more replies)
  0 siblings, 5 replies; 6+ messages in thread
From: ailiop-git @ 2021-03-10 18:37 UTC (permalink / raw)
  To: ml

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

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

https://github.com/ailiop-git/void-packages cyrus-sasl
https://github.com/void-linux/void-packages/pull/29383

cyrus-sasl-modules: remove dependency to main package
The various sasl modules do not dependent on any of the tools and
daemons that are included in the main cyrus-sasl package. Drop the
dependency.

<!-- Mark items with [x] where applicable -->

#### General
- [ ] This is a new package and it conforms to the [quality requirements](https://github.com/void-linux/void-packages/blob/master/Manual.md#quality-requirements)

#### Have the results of the proposed changes been tested?
- [x] I use the packages affected by the proposed changes on a regular basis and confirm this PR works for me
- [ ] I generally don't use the affected packages but briefly tested this PR

<!--
If GitHub CI cannot be used to validate the build result (for example, if the
build is likely to take several hours), make sure to
[skip CI](https://github.com/void-linux/void-packages/blob/master/CONTRIBUTING.md#continuous-integration).
When skipping CI, uncomment and fill out the following section.
Note: for builds that are likely to complete in less than 2 hours, it is not
acceptable to skip CI.
-->
<!-- 
#### Does it build and run successfully? 
(Please choose at least one native build and, if supported, at least one cross build. More are better.)
- [ ] 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/29383.patch is attached

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

From 984b3f136ce18f35c38245ac1d8025e3ed467abe Mon Sep 17 00:00:00 2001
From: Anthony Iliopoulos <ailiop@altatus.com>
Date: Wed, 10 Mar 2021 19:34:59 +0100
Subject: [PATCH] cyrus-sasl-modules: remove dependency to main package

The various sasl modules do not dependent on any of the tools and
daemons that are included in the main cyrus-sasl package. Drop the
dependency.
---
 srcpkgs/cyrus-sasl/template | 4 ----
 1 file changed, 4 deletions(-)

diff --git a/srcpkgs/cyrus-sasl/template b/srcpkgs/cyrus-sasl/template
index 585ac7a0e94..5f863227ef5 100644
--- a/srcpkgs/cyrus-sasl/template
+++ b/srcpkgs/cyrus-sasl/template
@@ -39,7 +39,6 @@ post_install() {
 }
 
 cyrus-sasl-modules-ldap_package() {
-	depends="cyrus-sasl>=${version}_${revision}"
 	short_desc="Cyrus SASL - pluggable authentication modules (LDAP)"
 	pkg_install() {
 		vmove "usr/lib/sasl2/libldapdb.*"
@@ -47,7 +46,6 @@ cyrus-sasl-modules-ldap_package() {
 }
 
 cyrus-sasl-modules-sql_package() {
-	depends="cyrus-sasl>=${version}_${revision}"
 	short_desc="Cyrus SASL - pluggable authentication modules (SQL)"
 	pkg_install() {
 		vmove "usr/lib/sasl2/libsql.*"
@@ -55,7 +53,6 @@ cyrus-sasl-modules-sql_package() {
 }
 
 cyrus-sasl-modules-gssapi_package() {
-	depends="cyrus-sasl>=${version}_${revision}"
 	short_desc="Cyrus SASL - pluggable authentication modules (GSSAPI)"
 	pkg_install() {
 		vmove "usr/lib/sasl2/libgssapi*"
@@ -63,7 +60,6 @@ cyrus-sasl-modules-gssapi_package() {
 }
 
 cyrus-sasl-modules_package() {
-	depends="cyrus-sasl>=${version}_${revision}"
 	short_desc="Cyrus SASL - pluggable authentication modules"
 	pkg_install() {
 		vmove usr/lib/sasl2

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

* Re: cyrus-sasl-modules: remove dependency to main package
  2021-03-10 18:37 [PR PATCH] cyrus-sasl-modules: remove dependency to main package ailiop-git
@ 2021-03-11 20:58 ` ericonr
  2021-03-11 21:05 ` [PR PATCH] [Updated] " ailiop-git
                   ` (3 subsequent siblings)
  4 siblings, 0 replies; 6+ messages in thread
From: ericonr @ 2021-03-11 20:58 UTC (permalink / raw)
  To: ml

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

New comment by ericonr on void-packages repository

https://github.com/void-linux/void-packages/pull/29383#issuecomment-797044475

Comment:
Lacks a revbump.

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

* Re: [PR PATCH] [Updated] cyrus-sasl-modules: remove dependency to main package
  2021-03-10 18:37 [PR PATCH] cyrus-sasl-modules: remove dependency to main package ailiop-git
  2021-03-11 20:58 ` ericonr
@ 2021-03-11 21:05 ` ailiop-git
  2021-03-11 21:09 ` ericonr
                   ` (2 subsequent siblings)
  4 siblings, 0 replies; 6+ messages in thread
From: ailiop-git @ 2021-03-11 21:05 UTC (permalink / raw)
  To: ml

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

There is an updated pull request by ailiop-git against master on the void-packages repository

https://github.com/ailiop-git/void-packages cyrus-sasl
https://github.com/void-linux/void-packages/pull/29383

cyrus-sasl-modules: remove dependency to main package
The various sasl modules do not dependent on any of the tools and
daemons that are included in the main cyrus-sasl package. Drop the
dependency.

<!-- Mark items with [x] where applicable -->

#### General
- [ ] This is a new package and it conforms to the [quality requirements](https://github.com/void-linux/void-packages/blob/master/Manual.md#quality-requirements)

#### Have the results of the proposed changes been tested?
- [x] I use the packages affected by the proposed changes on a regular basis and confirm this PR works for me
- [ ] I generally don't use the affected packages but briefly tested this PR

<!--
If GitHub CI cannot be used to validate the build result (for example, if the
build is likely to take several hours), make sure to
[skip CI](https://github.com/void-linux/void-packages/blob/master/CONTRIBUTING.md#continuous-integration).
When skipping CI, uncomment and fill out the following section.
Note: for builds that are likely to complete in less than 2 hours, it is not
acceptable to skip CI.
-->
<!-- 
#### Does it build and run successfully? 
(Please choose at least one native build and, if supported, at least one cross build. More are better.)
- [ ] 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/29383.patch is attached

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

From 7d0503c017491f3a3202e0452c28ba07b76dfd19 Mon Sep 17 00:00:00 2001
From: Anthony Iliopoulos <ailiop@altatus.com>
Date: Wed, 10 Mar 2021 19:34:59 +0100
Subject: [PATCH] cyrus-sasl-modules: remove dependency to main package

The various sasl modules do not dependent on any of the tools and
daemons that are included in the main cyrus-sasl package. Drop the
dependency.
---
 srcpkgs/cyrus-sasl/template | 6 +-----
 1 file changed, 1 insertion(+), 5 deletions(-)

diff --git a/srcpkgs/cyrus-sasl/template b/srcpkgs/cyrus-sasl/template
index 585ac7a0e94..608049a1d88 100644
--- a/srcpkgs/cyrus-sasl/template
+++ b/srcpkgs/cyrus-sasl/template
@@ -1,7 +1,7 @@
 # Template file for 'cyrus-sasl'
 pkgname=cyrus-sasl
 version=2.1.27
-revision=6
+revision=7
 build_style=gnu-configure
 configure_args="--disable-static --enable-shared --enable-checkapop
  --enable-cram --enable-digest --disable-otp --disable-srp
@@ -39,7 +39,6 @@ post_install() {
 }
 
 cyrus-sasl-modules-ldap_package() {
-	depends="cyrus-sasl>=${version}_${revision}"
 	short_desc="Cyrus SASL - pluggable authentication modules (LDAP)"
 	pkg_install() {
 		vmove "usr/lib/sasl2/libldapdb.*"
@@ -47,7 +46,6 @@ cyrus-sasl-modules-ldap_package() {
 }
 
 cyrus-sasl-modules-sql_package() {
-	depends="cyrus-sasl>=${version}_${revision}"
 	short_desc="Cyrus SASL - pluggable authentication modules (SQL)"
 	pkg_install() {
 		vmove "usr/lib/sasl2/libsql.*"
@@ -55,7 +53,6 @@ cyrus-sasl-modules-sql_package() {
 }
 
 cyrus-sasl-modules-gssapi_package() {
-	depends="cyrus-sasl>=${version}_${revision}"
 	short_desc="Cyrus SASL - pluggable authentication modules (GSSAPI)"
 	pkg_install() {
 		vmove "usr/lib/sasl2/libgssapi*"
@@ -63,7 +60,6 @@ cyrus-sasl-modules-gssapi_package() {
 }
 
 cyrus-sasl-modules_package() {
-	depends="cyrus-sasl>=${version}_${revision}"
 	short_desc="Cyrus SASL - pluggable authentication modules"
 	pkg_install() {
 		vmove usr/lib/sasl2

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

* Re: cyrus-sasl-modules: remove dependency to main package
  2021-03-10 18:37 [PR PATCH] cyrus-sasl-modules: remove dependency to main package ailiop-git
  2021-03-11 20:58 ` ericonr
  2021-03-11 21:05 ` [PR PATCH] [Updated] " ailiop-git
@ 2021-03-11 21:09 ` ericonr
  2021-03-11 21:11 ` [PR PATCH] [Updated] " ailiop-git
  2021-04-11  4:07 ` [PR PATCH] [Closed]: " ericonr
  4 siblings, 0 replies; 6+ messages in thread
From: ericonr @ 2021-03-11 21:09 UTC (permalink / raw)
  To: ml

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

New comment by ericonr on void-packages repository

https://github.com/void-linux/void-packages/pull/29383#issuecomment-797050698

Comment:
I think the preferred commit title would be `cyrus-sasl: remove dependency to main package from modules`.

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

* Re: [PR PATCH] [Updated] cyrus-sasl-modules: remove dependency to main package
  2021-03-10 18:37 [PR PATCH] cyrus-sasl-modules: remove dependency to main package ailiop-git
                   ` (2 preceding siblings ...)
  2021-03-11 21:09 ` ericonr
@ 2021-03-11 21:11 ` ailiop-git
  2021-04-11  4:07 ` [PR PATCH] [Closed]: " ericonr
  4 siblings, 0 replies; 6+ messages in thread
From: ailiop-git @ 2021-03-11 21:11 UTC (permalink / raw)
  To: ml

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

There is an updated pull request by ailiop-git against master on the void-packages repository

https://github.com/ailiop-git/void-packages cyrus-sasl
https://github.com/void-linux/void-packages/pull/29383

cyrus-sasl-modules: remove dependency to main package
The various sasl modules do not dependent on any of the tools and
daemons that are included in the main cyrus-sasl package. Drop the
dependency.

<!-- Mark items with [x] where applicable -->

#### General
- [ ] This is a new package and it conforms to the [quality requirements](https://github.com/void-linux/void-packages/blob/master/Manual.md#quality-requirements)

#### Have the results of the proposed changes been tested?
- [x] I use the packages affected by the proposed changes on a regular basis and confirm this PR works for me
- [ ] I generally don't use the affected packages but briefly tested this PR

<!--
If GitHub CI cannot be used to validate the build result (for example, if the
build is likely to take several hours), make sure to
[skip CI](https://github.com/void-linux/void-packages/blob/master/CONTRIBUTING.md#continuous-integration).
When skipping CI, uncomment and fill out the following section.
Note: for builds that are likely to complete in less than 2 hours, it is not
acceptable to skip CI.
-->
<!-- 
#### Does it build and run successfully? 
(Please choose at least one native build and, if supported, at least one cross build. More are better.)
- [ ] 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/29383.patch is attached

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

From 9bff85fe4c4d3ba4a8bdd3f79f0944eefc3201e7 Mon Sep 17 00:00:00 2001
From: Anthony Iliopoulos <ailiop@altatus.com>
Date: Wed, 10 Mar 2021 19:34:59 +0100
Subject: [PATCH] cyrus-sasl: remove dependency to main package from modules

The various sasl modules do not dependent on any of the tools and
daemons that are included in the main cyrus-sasl package. Drop the
dependency.
---
 srcpkgs/cyrus-sasl/template | 6 +-----
 1 file changed, 1 insertion(+), 5 deletions(-)

diff --git a/srcpkgs/cyrus-sasl/template b/srcpkgs/cyrus-sasl/template
index 585ac7a0e94..608049a1d88 100644
--- a/srcpkgs/cyrus-sasl/template
+++ b/srcpkgs/cyrus-sasl/template
@@ -1,7 +1,7 @@
 # Template file for 'cyrus-sasl'
 pkgname=cyrus-sasl
 version=2.1.27
-revision=6
+revision=7
 build_style=gnu-configure
 configure_args="--disable-static --enable-shared --enable-checkapop
  --enable-cram --enable-digest --disable-otp --disable-srp
@@ -39,7 +39,6 @@ post_install() {
 }
 
 cyrus-sasl-modules-ldap_package() {
-	depends="cyrus-sasl>=${version}_${revision}"
 	short_desc="Cyrus SASL - pluggable authentication modules (LDAP)"
 	pkg_install() {
 		vmove "usr/lib/sasl2/libldapdb.*"
@@ -47,7 +46,6 @@ cyrus-sasl-modules-ldap_package() {
 }
 
 cyrus-sasl-modules-sql_package() {
-	depends="cyrus-sasl>=${version}_${revision}"
 	short_desc="Cyrus SASL - pluggable authentication modules (SQL)"
 	pkg_install() {
 		vmove "usr/lib/sasl2/libsql.*"
@@ -55,7 +53,6 @@ cyrus-sasl-modules-sql_package() {
 }
 
 cyrus-sasl-modules-gssapi_package() {
-	depends="cyrus-sasl>=${version}_${revision}"
 	short_desc="Cyrus SASL - pluggable authentication modules (GSSAPI)"
 	pkg_install() {
 		vmove "usr/lib/sasl2/libgssapi*"
@@ -63,7 +60,6 @@ cyrus-sasl-modules-gssapi_package() {
 }
 
 cyrus-sasl-modules_package() {
-	depends="cyrus-sasl>=${version}_${revision}"
 	short_desc="Cyrus SASL - pluggable authentication modules"
 	pkg_install() {
 		vmove usr/lib/sasl2

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

* Re: [PR PATCH] [Closed]: cyrus-sasl-modules: remove dependency to main package
  2021-03-10 18:37 [PR PATCH] cyrus-sasl-modules: remove dependency to main package ailiop-git
                   ` (3 preceding siblings ...)
  2021-03-11 21:11 ` [PR PATCH] [Updated] " ailiop-git
@ 2021-04-11  4:07 ` ericonr
  4 siblings, 0 replies; 6+ messages in thread
From: ericonr @ 2021-04-11  4:07 UTC (permalink / raw)
  To: ml

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

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

cyrus-sasl-modules: remove dependency to main package
https://github.com/void-linux/void-packages/pull/29383

Description:
The various sasl modules do not dependent on any of the tools and
daemons that are included in the main cyrus-sasl package. Drop the
dependency.

<!-- Mark items with [x] where applicable -->

#### General
- [ ] This is a new package and it conforms to the [quality requirements](https://github.com/void-linux/void-packages/blob/master/Manual.md#quality-requirements)

#### Have the results of the proposed changes been tested?
- [x] I use the packages affected by the proposed changes on a regular basis and confirm this PR works for me
- [ ] I generally don't use the affected packages but briefly tested this PR

<!--
If GitHub CI cannot be used to validate the build result (for example, if the
build is likely to take several hours), make sure to
[skip CI](https://github.com/void-linux/void-packages/blob/master/CONTRIBUTING.md#continuous-integration).
When skipping CI, uncomment and fill out the following section.
Note: for builds that are likely to complete in less than 2 hours, it is not
acceptable to skip CI.
-->
<!-- 
#### Does it build and run successfully? 
(Please choose at least one native build and, if supported, at least one cross build. More are better.)
- [ ] 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] 6+ messages in thread

end of thread, other threads:[~2021-04-11  4:07 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-03-10 18:37 [PR PATCH] cyrus-sasl-modules: remove dependency to main package ailiop-git
2021-03-11 20:58 ` ericonr
2021-03-11 21:05 ` [PR PATCH] [Updated] " ailiop-git
2021-03-11 21:09 ` ericonr
2021-03-11 21:11 ` [PR PATCH] [Updated] " ailiop-git
2021-04-11  4:07 ` [PR PATCH] [Closed]: " 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).