* [PR PATCH] tectonic: remove hardcoded versions
@ 2019-09-14 23:46 voidlinux-github
2019-09-15 12:00 ` voidlinux-github
` (8 more replies)
0 siblings, 9 replies; 10+ messages in thread
From: voidlinux-github @ 2019-09-14 23:46 UTC (permalink / raw)
To: ml
[-- Attachment #1: Type: text/plain, Size: 490 bytes --]
There is a new pull request by concatime against master on the void-packages repository
https://github.com/concatime/void-packages tectonic-openssl-fix
https://github.com/void-linux/void-packages/pull/14463
tectonic: remove hardcoded versions
This way, we donât need to change theses numbers at each breaking change from libressl,
because cargo will pick the latest openssl and openssl-sys.
A patch file from https://github.com/void-linux/void-packages/pull/14463.patch is attached
[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: github-pr-tectonic-openssl-fix-14463.patch --]
[-- Type: text/x-diff, Size: 1500 bytes --]
From 0686bf9d2052da292d37afa479e19dd12de64967 Mon Sep 17 00:00:00 2001
From: Issam Maghni <me@concati.me>
Date: Sat, 14 Sep 2019 19:45:47 -0400
Subject: [PATCH] tectonic: remove hardcoded versions
---
srcpkgs/tectonic/template | 13 +++++++++++--
1 file changed, 11 insertions(+), 2 deletions(-)
diff --git a/srcpkgs/tectonic/template b/srcpkgs/tectonic/template
index 19954ca6d7a..effe33ce097 100644
--- a/srcpkgs/tectonic/template
+++ b/srcpkgs/tectonic/template
@@ -13,9 +13,18 @@ changelog="https://raw.githubusercontent.com/tectonic-typesetting/tectonic/maste
distfiles="https://github.com/tectonic-typesetting/${pkgname}/archive/v${version}.tar.gz"
checksum=e700dc691dfd092adfe098b716992136343ddfac5eaabb1e8cfae4e63f8454c7
+# REMOVE THIS SECTION ON NEXT VERSION
pre_build() {
- cargo update --package openssl-sys --precise 0.9.46
- cargo update --package openssl --precise 0.10.22
+ # openssl-sys v0.9.39
+ # This crate is only compatible with OpenSSL 1.0.1 through 1.1.1, or LibreSSL 2.5
+ # through 2.8.1, but a different version of OpenSSL was found. The build is now aborting
+ # due to this version mismatch.
+ # openssl 0.10.15
+ # error[E0432]: unresolved imports `ffi::SSLv23_method`, `ffi::DTLSv1_method`
+ # error[E0432]: unresolved imports `ffi::SSLEAY_VERSION`, `ffi::SSLEAY_CFLAGS`,
+ # `ffi::SSLEAY_BUILT_ON`, `ffi::SSLEAY_PLATFORM`, `ffi::SSLEAY_DIR`, `ffi::SSLeay`,
+ # `ffi::SSLeay_version`
+ cargo update --package openssl --package openssl-sys
}
post_install() {
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: tectonic: remove hardcoded versions
2019-09-14 23:46 [PR PATCH] tectonic: remove hardcoded versions voidlinux-github
@ 2019-09-15 12:00 ` voidlinux-github
2019-09-15 15:31 ` voidlinux-github
` (7 subsequent siblings)
8 siblings, 0 replies; 10+ messages in thread
From: voidlinux-github @ 2019-09-15 12:00 UTC (permalink / raw)
To: ml
[-- Attachment #1: Type: text/plain, Size: 237 bytes --]
New comment by Duncaen on void-packages repository
https://github.com/void-linux/void-packages/pull/14463#issuecomment-531559709
Comment:
Not sure if this is a good idea. Optimally all versions are locked so a package is reproducible.
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: tectonic: remove hardcoded versions
2019-09-14 23:46 [PR PATCH] tectonic: remove hardcoded versions voidlinux-github
2019-09-15 12:00 ` voidlinux-github
@ 2019-09-15 15:31 ` voidlinux-github
2019-09-15 15:45 ` [PR PATCH] [Updated] " voidlinux-github
` (6 subsequent siblings)
8 siblings, 0 replies; 10+ messages in thread
From: voidlinux-github @ 2019-09-15 15:31 UTC (permalink / raw)
To: ml
[-- Attachment #1: Type: text/plain, Size: 255 bytes --]
New comment by concatime on void-packages repository
https://github.com/void-linux/void-packages/pull/14463#issuecomment-531575146
Comment:
Asking on IRC, it seems that `cargo update` will change only the patch number (X.Y.PATCH), so it should be safe.
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PR PATCH] [Updated] tectonic: remove hardcoded versions
2019-09-14 23:46 [PR PATCH] tectonic: remove hardcoded versions voidlinux-github
` (2 preceding siblings ...)
2019-09-15 15:45 ` [PR PATCH] [Updated] " voidlinux-github
@ 2019-09-15 15:45 ` voidlinux-github
2019-09-15 16:07 ` voidlinux-github
` (4 subsequent siblings)
8 siblings, 0 replies; 10+ messages in thread
From: voidlinux-github @ 2019-09-15 15:45 UTC (permalink / raw)
To: ml
[-- Attachment #1: Type: text/plain, Size: 495 bytes --]
There is an updated pull request by concatime against master on the void-packages repository
https://github.com/concatime/void-packages tectonic-openssl-fix
https://github.com/void-linux/void-packages/pull/14463
tectonic: remove hardcoded versions
This way, we donât need to change theses numbers at each breaking change from libressl,
because cargo will pick the latest openssl and openssl-sys.
A patch file from https://github.com/void-linux/void-packages/pull/14463.patch is attached
[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: github-pr-tectonic-openssl-fix-14463.patch --]
[-- Type: text/x-diff, Size: 2738 bytes --]
From 0686bf9d2052da292d37afa479e19dd12de64967 Mon Sep 17 00:00:00 2001
From: Issam Maghni <me@concati.me>
Date: Sat, 14 Sep 2019 19:45:47 -0400
Subject: [PATCH 1/2] tectonic: remove hardcoded versions
---
srcpkgs/tectonic/template | 13 +++++++++++--
1 file changed, 11 insertions(+), 2 deletions(-)
diff --git a/srcpkgs/tectonic/template b/srcpkgs/tectonic/template
index 19954ca6d7a..effe33ce097 100644
--- a/srcpkgs/tectonic/template
+++ b/srcpkgs/tectonic/template
@@ -13,9 +13,18 @@ changelog="https://raw.githubusercontent.com/tectonic-typesetting/tectonic/maste
distfiles="https://github.com/tectonic-typesetting/${pkgname}/archive/v${version}.tar.gz"
checksum=e700dc691dfd092adfe098b716992136343ddfac5eaabb1e8cfae4e63f8454c7
+# REMOVE THIS SECTION ON NEXT VERSION
pre_build() {
- cargo update --package openssl-sys --precise 0.9.46
- cargo update --package openssl --precise 0.10.22
+ # openssl-sys v0.9.39
+ # This crate is only compatible with OpenSSL 1.0.1 through 1.1.1, or LibreSSL 2.5
+ # through 2.8.1, but a different version of OpenSSL was found. The build is now aborting
+ # due to this version mismatch.
+ # openssl 0.10.15
+ # error[E0432]: unresolved imports `ffi::SSLv23_method`, `ffi::DTLSv1_method`
+ # error[E0432]: unresolved imports `ffi::SSLEAY_VERSION`, `ffi::SSLEAY_CFLAGS`,
+ # `ffi::SSLEAY_BUILT_ON`, `ffi::SSLEAY_PLATFORM`, `ffi::SSLEAY_DIR`, `ffi::SSLeay`,
+ # `ffi::SSLeay_version`
+ cargo update --package openssl --package openssl-sys
}
post_install() {
From 56e8420132969e50b3355ecffcf28d74a09bf8ef Mon Sep 17 00:00:00 2001
From: Issam Maghni <me@concati.me>
Date: Sun, 15 Sep 2019 11:46:32 -0400
Subject: [PATCH 2/2] tectonic: xlint fix false-positive
---
srcpkgs/tectonic/template | 8 ++++----
1 file changed, 4 insertions(+), 4 deletions(-)
diff --git a/srcpkgs/tectonic/template b/srcpkgs/tectonic/template
index effe33ce097..619616b50db 100644
--- a/srcpkgs/tectonic/template
+++ b/srcpkgs/tectonic/template
@@ -20,10 +20,10 @@ pre_build() {
# through 2.8.1, but a different version of OpenSSL was found. The build is now aborting
# due to this version mismatch.
# openssl 0.10.15
- # error[E0432]: unresolved imports `ffi::SSLv23_method`, `ffi::DTLSv1_method`
- # error[E0432]: unresolved imports `ffi::SSLEAY_VERSION`, `ffi::SSLEAY_CFLAGS`,
- # `ffi::SSLEAY_BUILT_ON`, `ffi::SSLEAY_PLATFORM`, `ffi::SSLEAY_DIR`, `ffi::SSLeay`,
- # `ffi::SSLeay_version`
+ # error[E0432]: unresolved imports ffi::SSLv23_method, ffi::DTLSv1_method
+ # error[E0432]: unresolved imports ffi::SSLEAY_VERSION, ffi::SSLEAY_CFLAGS,
+ # ffi::SSLEAY_BUILT_ON, ffi::SSLEAY_PLATFORM, ffi::SSLEAY_DIR, ffi::SSLeay,
+ # ffi::SSLeay_version
cargo update --package openssl --package openssl-sys
}
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PR PATCH] [Updated] tectonic: remove hardcoded versions
2019-09-14 23:46 [PR PATCH] tectonic: remove hardcoded versions voidlinux-github
2019-09-15 12:00 ` voidlinux-github
2019-09-15 15:31 ` voidlinux-github
@ 2019-09-15 15:45 ` voidlinux-github
2019-09-15 15:45 ` voidlinux-github
` (5 subsequent siblings)
8 siblings, 0 replies; 10+ messages in thread
From: voidlinux-github @ 2019-09-15 15:45 UTC (permalink / raw)
To: ml
[-- Attachment #1: Type: text/plain, Size: 495 bytes --]
There is an updated pull request by concatime against master on the void-packages repository
https://github.com/concatime/void-packages tectonic-openssl-fix
https://github.com/void-linux/void-packages/pull/14463
tectonic: remove hardcoded versions
This way, we donât need to change theses numbers at each breaking change from libressl,
because cargo will pick the latest openssl and openssl-sys.
A patch file from https://github.com/void-linux/void-packages/pull/14463.patch is attached
[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: github-pr-tectonic-openssl-fix-14463.patch --]
[-- Type: text/x-diff, Size: 2738 bytes --]
From 0686bf9d2052da292d37afa479e19dd12de64967 Mon Sep 17 00:00:00 2001
From: Issam Maghni <me@concati.me>
Date: Sat, 14 Sep 2019 19:45:47 -0400
Subject: [PATCH 1/2] tectonic: remove hardcoded versions
---
srcpkgs/tectonic/template | 13 +++++++++++--
1 file changed, 11 insertions(+), 2 deletions(-)
diff --git a/srcpkgs/tectonic/template b/srcpkgs/tectonic/template
index 19954ca6d7a..effe33ce097 100644
--- a/srcpkgs/tectonic/template
+++ b/srcpkgs/tectonic/template
@@ -13,9 +13,18 @@ changelog="https://raw.githubusercontent.com/tectonic-typesetting/tectonic/maste
distfiles="https://github.com/tectonic-typesetting/${pkgname}/archive/v${version}.tar.gz"
checksum=e700dc691dfd092adfe098b716992136343ddfac5eaabb1e8cfae4e63f8454c7
+# REMOVE THIS SECTION ON NEXT VERSION
pre_build() {
- cargo update --package openssl-sys --precise 0.9.46
- cargo update --package openssl --precise 0.10.22
+ # openssl-sys v0.9.39
+ # This crate is only compatible with OpenSSL 1.0.1 through 1.1.1, or LibreSSL 2.5
+ # through 2.8.1, but a different version of OpenSSL was found. The build is now aborting
+ # due to this version mismatch.
+ # openssl 0.10.15
+ # error[E0432]: unresolved imports `ffi::SSLv23_method`, `ffi::DTLSv1_method`
+ # error[E0432]: unresolved imports `ffi::SSLEAY_VERSION`, `ffi::SSLEAY_CFLAGS`,
+ # `ffi::SSLEAY_BUILT_ON`, `ffi::SSLEAY_PLATFORM`, `ffi::SSLEAY_DIR`, `ffi::SSLeay`,
+ # `ffi::SSLeay_version`
+ cargo update --package openssl --package openssl-sys
}
post_install() {
From 56e8420132969e50b3355ecffcf28d74a09bf8ef Mon Sep 17 00:00:00 2001
From: Issam Maghni <me@concati.me>
Date: Sun, 15 Sep 2019 11:46:32 -0400
Subject: [PATCH 2/2] tectonic: xlint fix false-positive
---
srcpkgs/tectonic/template | 8 ++++----
1 file changed, 4 insertions(+), 4 deletions(-)
diff --git a/srcpkgs/tectonic/template b/srcpkgs/tectonic/template
index effe33ce097..619616b50db 100644
--- a/srcpkgs/tectonic/template
+++ b/srcpkgs/tectonic/template
@@ -20,10 +20,10 @@ pre_build() {
# through 2.8.1, but a different version of OpenSSL was found. The build is now aborting
# due to this version mismatch.
# openssl 0.10.15
- # error[E0432]: unresolved imports `ffi::SSLv23_method`, `ffi::DTLSv1_method`
- # error[E0432]: unresolved imports `ffi::SSLEAY_VERSION`, `ffi::SSLEAY_CFLAGS`,
- # `ffi::SSLEAY_BUILT_ON`, `ffi::SSLEAY_PLATFORM`, `ffi::SSLEAY_DIR`, `ffi::SSLeay`,
- # `ffi::SSLeay_version`
+ # error[E0432]: unresolved imports ffi::SSLv23_method, ffi::DTLSv1_method
+ # error[E0432]: unresolved imports ffi::SSLEAY_VERSION, ffi::SSLEAY_CFLAGS,
+ # ffi::SSLEAY_BUILT_ON, ffi::SSLEAY_PLATFORM, ffi::SSLEAY_DIR, ffi::SSLeay,
+ # ffi::SSLeay_version
cargo update --package openssl --package openssl-sys
}
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: tectonic: remove hardcoded versions
2019-09-14 23:46 [PR PATCH] tectonic: remove hardcoded versions voidlinux-github
` (3 preceding siblings ...)
2019-09-15 15:45 ` voidlinux-github
@ 2019-09-15 16:07 ` voidlinux-github
2019-09-15 16:10 ` voidlinux-github
` (3 subsequent siblings)
8 siblings, 0 replies; 10+ messages in thread
From: voidlinux-github @ 2019-09-15 16:07 UTC (permalink / raw)
To: ml
[-- Attachment #1: Type: text/plain, Size: 680 bytes --]
New comment by Duncaen on void-packages repository
https://github.com/void-linux/void-packages/pull/14463#issuecomment-531577954
Comment:
cargo packages usually comes with `Cargo.lock`, which makes sure it is using a fixed version.
We don't run `cargo update` each build, resulting in building the same code each time the revision is bumped. If you start selectively updating some random dependencies to the latest revision available you get different binaries every time the package is build.
Checking out a version of a void template should optimally produce exactly the same binaries as the previous build, the build date shouldn't have influence on the produced binaries.
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: tectonic: remove hardcoded versions
2019-09-14 23:46 [PR PATCH] tectonic: remove hardcoded versions voidlinux-github
` (4 preceding siblings ...)
2019-09-15 16:07 ` voidlinux-github
@ 2019-09-15 16:10 ` voidlinux-github
2019-09-15 16:11 ` voidlinux-github
` (2 subsequent siblings)
8 siblings, 0 replies; 10+ messages in thread
From: voidlinux-github @ 2019-09-15 16:10 UTC (permalink / raw)
To: ml
[-- Attachment #1: Type: text/plain, Size: 679 bytes --]
New comment by Duncaen on void-packages repository
https://github.com/void-linux/void-packages/pull/14463#issuecomment-531577954
Comment:
cargo packages usually come with `Cargo.lock`, which makes sure it is using a fixed version.
We don't run `cargo update` each build, resulting in building the same code each time the revision is bumped. If you start selectively updating some random dependencies to the latest revision available you get different binaries every time the package is build.
Checking out a version of a void template should optimally produce exactly the same binaries as the previous build, the build date shouldn't have influence on the produced binaries.
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: tectonic: remove hardcoded versions
2019-09-14 23:46 [PR PATCH] tectonic: remove hardcoded versions voidlinux-github
` (5 preceding siblings ...)
2019-09-15 16:10 ` voidlinux-github
@ 2019-09-15 16:11 ` voidlinux-github
2019-09-15 20:18 ` [PR PATCH] [Closed]: " voidlinux-github
2019-09-15 20:18 ` voidlinux-github
8 siblings, 0 replies; 10+ messages in thread
From: voidlinux-github @ 2019-09-15 16:11 UTC (permalink / raw)
To: ml
[-- Attachment #1: Type: text/plain, Size: 684 bytes --]
New comment by Duncaen on void-packages repository
https://github.com/void-linux/void-packages/pull/14463#issuecomment-531577954
Comment:
cargo packages usually come with `Cargo.lock`, which makes sure it is using a fixed version.
We don't run `cargo update` each build, resulting in building the same code each time the revision is bumped. If you start selectively updating some random dependencies to the latest minor version available you get different binaries every time the package is build.
Checking out a version of a void template should optimally produce exactly the same binaries as the previous build, the build date shouldn't have influence on the produced binaries.
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: tectonic: remove hardcoded versions
2019-09-14 23:46 [PR PATCH] tectonic: remove hardcoded versions voidlinux-github
` (7 preceding siblings ...)
2019-09-15 20:18 ` [PR PATCH] [Closed]: " voidlinux-github
@ 2019-09-15 20:18 ` voidlinux-github
8 siblings, 0 replies; 10+ messages in thread
From: voidlinux-github @ 2019-09-15 20:18 UTC (permalink / raw)
To: ml
[-- Attachment #1: Type: text/plain, Size: 403 bytes --]
New comment by concatime on void-packages repository
https://github.com/void-linux/void-packages/pull/14463#issuecomment-531595970
Comment:
> If you start selectively updating some random dependencies
The two pacakges (openssl and openssl-sys) are not randomly selected. We cannot build this package w/o a version bump of these two. That being said, feel free to do whatever you want this this PR.
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PR PATCH] [Closed]: tectonic: remove hardcoded versions
2019-09-14 23:46 [PR PATCH] tectonic: remove hardcoded versions voidlinux-github
` (6 preceding siblings ...)
2019-09-15 16:11 ` voidlinux-github
@ 2019-09-15 20:18 ` voidlinux-github
2019-09-15 20:18 ` voidlinux-github
8 siblings, 0 replies; 10+ messages in thread
From: voidlinux-github @ 2019-09-15 20:18 UTC (permalink / raw)
To: ml
[-- Attachment #1: Type: text/plain, Size: 322 bytes --]
There's a closed pull request on the void-packages repository
tectonic: remove hardcoded versions
https://github.com/void-linux/void-packages/pull/14463
Description:
This way, we donât need to change theses numbers at each breaking change from libressl,
because cargo will pick the latest openssl and openssl-sys.
^ permalink raw reply [flat|nested] 10+ messages in thread
end of thread, other threads:[~2019-09-15 20:18 UTC | newest]
Thread overview: 10+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-09-14 23:46 [PR PATCH] tectonic: remove hardcoded versions voidlinux-github
2019-09-15 12:00 ` voidlinux-github
2019-09-15 15:31 ` voidlinux-github
2019-09-15 15:45 ` [PR PATCH] [Updated] " voidlinux-github
2019-09-15 15:45 ` voidlinux-github
2019-09-15 16:07 ` voidlinux-github
2019-09-15 16:10 ` voidlinux-github
2019-09-15 16:11 ` voidlinux-github
2019-09-15 20:18 ` [PR PATCH] [Closed]: " voidlinux-github
2019-09-15 20:18 ` voidlinux-github
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).