Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Wrong format for downloaded files for release 0.8.4 #104

Closed
ellert opened this issue Feb 2, 2023 · 5 comments
Closed

Wrong format for downloaded files for release 0.8.4 #104

ellert opened this issue Feb 2, 2023 · 5 comments

Comments

@ellert
Copy link
Contributor

ellert commented Feb 2, 2023

The download link on the release page does not result in a file of the correct format.
If I download:

$ wget https://github.com/cern-fts/davix/releases/download/R_0_8_4/davix-0.8.4.tar.gz

The retrieved file is a zip file, not a gzipped tar archive.

$ file davix-0.8.4.tar.gz
davix-0.8.4.tar.gz: Zip archive data, at least v2.0 to extract, compression method=deflate

If I inflate this file

$ mkdir z
$ cd z
$ unzip ../davix-0.8.4.tar.gz
Archive: ../davix-0.8.4.tar.gz
inflating: davix-0.8.4.tar.gz
$ file davix-0.8.4.tar.gz
davix-0.8.4.tar.gz: gzip compressed data, last modified: Thu Feb 2 11:17:28 2023, from Unix, original size modulo 2^32 36597760

I get the correct file. So the link give a zipped gzipped tar archive instead of a gzipped archive.

The corresponding link for the previous release gives the right format.

$ wget https://github.com/cern-fts/davix/releases/download/R_0_8_3/davix-0.8.3.tar.gz

$ file davix-0.8.3.tar.gz
davix-0.8.3.tar.gz: gzip compressed data, last modified: Tue Sep 13 10:19:00 2022, from Unix, original size modulo 2^32 36587520

(The davix-0.8.4-el7.x86_64.tar.gz link also gives the wrong format.)

chrisburr added a commit to regro-cf-autotick-bot/davix-feedstock that referenced this issue Feb 5, 2023
@amadio
Copy link
Contributor

amadio commented Feb 7, 2023

@joaopblopes This is blocking packaging of davix on several places (I also see a problem in Gentoo). Could you please upload the correct tarball replacing the zip file? Thanks!

@joaopblopes
Copy link
Contributor

joaopblopes commented Feb 7, 2023

Hi @amadio ,

It should be in the right format now.

$ wget https://github.com/cern-fts/davix/releases/download/R_0_8_4/davix-0.8.4.tar.gz

$ file davix-0.8.4.tar.gz
davix-0.8.4.tar.gz: gzip compressed data, last modified: Thu Feb 2 11:17:28 2023, from Unix, original size modulo 2^32 36597760

And:

$ wget https://github.com/cern-fts/davix/releases/download/R_0_8_4/davix-0.8.4-el7.x86_64.tar.gz

$ file davix-0.8.4-el7.x86_64.tar.gz
davix-0.8.4-el7.x86_64.tar.gz: gzip compressed data, last modified: Thu Feb 2 11:22:22 2023, from Unix, original size modulo 2^32 7188480

@amadio
Copy link
Contributor

amadio commented Feb 7, 2023

Thanks! Now it works correclty, I am just seeing some bad warnings with GCC 12 which may warrant fixing for the next release:

 * QA Notice: Package triggers severe warnings which indicate that it
 *            may exhibit random runtime failures.
 * /tmp/portage/net-libs/davix-0.8.4/work/davix-0.8.4/deps/libneon/src/ne_openssl.c:1332:39: warning: the comparison will always evaluate as ‘false’ for the address of ‘thread_lock_neon’ will never be NULL [-Waddress]
 * /usr/lib/gcc/x86_64-pc-linux-gnu/12/include/g++-v12/bits/basic_string.h:234:28: warning: array subscript 0 is outside array bounds of ‘void [8]’ [-Warray-bounds]
 * /usr/lib/gcc/x86_64-pc-linux-gnu/12/include/g++-v12/bits/basic_string.h:1071:16: warning: array subscript 0 is outside array bounds of ‘void [8]’ [-Warray-bounds]

@joaopblopes
Copy link
Contributor

Glad that it works now! Thanks for reporting this!

Regarding the warnings, I can have a look into them with Mihai. Maybe you can create a new issue to track it.

I'll close this issue here.

Cheers,
João

@chenrui333
Copy link

Thanks @joaopblopes!

chrisburr added a commit to regro-cf-autotick-bot/davix-feedstock that referenced this issue Mar 21, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants