Github messages for voidlinux
 help / color / mirror / Atom feed
* [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
  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: [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: 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 ` voidlinux-github
  2019-09-15 20:18 ` [PR PATCH] [Closed]: " 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
                   ` (6 preceding siblings ...)
  2019-09-15 16:11 ` voidlinux-github
@ 2019-09-15 20:18 ` voidlinux-github
  2019-09-15 20:18 ` [PR PATCH] [Closed]: " 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
                   ` (7 preceding siblings ...)
  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 ` voidlinux-github
2019-09-15 20:18 ` [PR PATCH] [Closed]: " 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).