public inbox for cygwin-apps@cygwin.com
 help / color / mirror / Atom feed
From: Marco Atzeri <marco.atzeri@gmail.com>
To: cygwin-apps@cygwin.com
Subject: Re: [Attn.Maintainer] libarchive
Date: Fri, 30 Oct 2020 11:23:12 +0100	[thread overview]
Message-ID: <b1a7d0e1-5c1d-6321-312f-519530ca1c87@gmail.com> (raw)
In-Reply-To: <87o8kovu0n.fsf@Rainer.invalid>

On 27.10.2020 11:32, Achim Gratz wrote:
> 
> Hi Marco,
> 
> the last release of libarchive hasn't been pushed to
> 
> https://cygwin.com/git-cygwin-packages/?p=git/cygwin-packages/libarchive.git
> 
> could that please be corrected?
> 
> Also it is built without ZStd and LZO support (the former presumably
> because you didn't have the headers installed and the latter because it
> requires a configure argument).  Looking into that I noticed that the
> configury rolls in Cygwin with MSys and MinGW and produces the expected
> nonsense, in particular the SSL library is not found that way.  It works
> slightly better via cmake, but that doesn't produce a correctly named
> DLL, so I stopped there.  Likewise there are tests for lrzip/grzip that
> don't seem to have a corresponding configure option (or configure
> tests), so I didn't try if they work.
> 
> Here are the fixed up cygport file plus the configury patch:
> 
> --8<---------------cut here---------------start------------->8---
> # -*- mode: sh; sh-shell: bash -*-
> # inherit cmake # doesn't produce a correctly named library
> BUILD_REQUIRES="  libbz2-devel zlib-devel libzstd-devel liblzo2-devel liblz4-devel liblzma-devel"
> BUILD_REQUIRES+=" libiconv-devel libxml2-devel libssl-devel"
> NAME="libarchive"
> VERSION="3.4.3"
> RELEASE="2"
> CATEGORY="Archive"
> SUMMARY="Multiple-format archive and compression library"
> DESCRIPTION="Libarchive is a programming library that can create and read
> several different streaming archive formats, including most popular tar
> variants, several cpio formats, and both BSD and GNU ar variants. It can also
> write shar archives and read ISO9660 CDROM images and ZIP archives. The bsdtar
> program is an implementation of tar(1) that is built on top of libarchive. It
> started as a test harness, but has grown into a feature-competitive replacement
> for GNU tar."
> HOMEPAGE="https://www.libarchive.org/"
> SRC_URI="https://www.libarchive.org/downloads/${P}.tar.gz"
> SRC_DIR="${P}"
> 
> PKG_NAMES="${PN}13 ${PN}-devel bsdtar bsdcpio bsdcat"
> eval ${PN}13_CONTENTS='"usr/bin/cygarchive-13.dll"'
> libarchive_devel_CONTENTS="usr/include/ usr/lib/ usr/share/doc/ usr/share/man/man[35]/"
> bsdtar_CONTENTS="usr/bin/bsdtar.exe usr/share/man/man1/bsdtar.*"
> bsdcpio_CONTENTS="usr/bin/bsdcpio.exe usr/share/man/man1/bsdcpio.*"
> bsdcat_CONTENTS="usr/bin/bsdcat.exe usr/share/man/man1/bsdcat.*"
> 
> CYGCONF_ARGS="--with-lzo2"
> --8<---------------cut here---------------end--------------->8---
> 
> --8<---------------cut here---------------start------------->8---
> --- origsrc/libarchive-3.4.3/configure.ac
> +++ src/libarchive-3.4.3/configure.ac
> @@ -86,13 +86,10 @@ AC_CANONICAL_HOST
>   
>   dnl Compilation on mingw and Cygwin needs special Makefile rules
>   inc_windows_files=no
> -inc_cygwin_files=no
>   case "$host_os" in
>     *mingw* ) inc_windows_files=yes ;;
> -  *cygwin* | *msys*) inc_cygwin_files=yes ;;
>   esac
>   AM_CONDITIONAL([INC_WINDOWS_FILES], [test $inc_windows_files = yes])
> -AM_CONDITIONAL([INC_CYGWIN_FILES], [test $inc_cygwin_files = yes])
>   
>   dnl Defines that are required for specific platforms (e.g. -D_POSIX_SOURCE, etc)
>   PLATFORMCPPFLAGS=
> @@ -243,7 +240,7 @@ AM_CONDITIONAL([STATIC_BSDCPIO], [ test
>   
>   # Set up defines needed before including any headers
>   case $host in
> -  *mingw* | *cygwin* | *msys*  )
> +  *mingw* | *msys*  )
>     AC_DEFINE([_WIN32_WINNT], 0x0502, [Define to '0x0502' for Windows Server 2003 APIs.])
>     AC_DEFINE([WINVER], 0x0502, [Define to '0x0502' for Windows Server 2003 APIs.])
>     AC_DEFINE([NTDDI_VERSION], 0x05020000, [Define to '0x05020000' for Windows Server 2003 APIs.])
> @@ -316,7 +313,7 @@ AC_ARG_WITH([bz2lib],
>   if test "x$with_bz2lib" != "xno"; then
>     AC_CHECK_HEADERS([bzlib.h])
>     case "$host_os" in
> -    *mingw* | *cygwin* | *msys*)
> +    *mingw* | *msys*)
>         dnl AC_CHECK_LIB cannot be used on the Windows port of libbz2, therefore
>   	  dnl use AC_LINK_IFELSE.
>   	  AC_MSG_CHECKING([for BZ2_bzDecompressInit in -lbz2])
> @@ -1089,7 +1086,7 @@ main(int argc, char **argv)
>   ])
>   
>   case "$host_os" in
> -  *mingw* | *cygwin* | *msys*)
> +  *mingw* | *msys*)
>   	;;
>     *)
>   	CRYPTO_CHECK(MD5, LIBC, md5)
> @@ -1157,7 +1154,7 @@ if test "x$with_openssl" != "xno"; then
>       AC_CHECK_HEADERS([openssl/evp.h])
>       saved_LIBS=$LIBS
>       case "$host_os" in
> -      *mingw* | *cygwin* | *msys*)
> +      *mingw* | *msys*)
>           case "$host_cpu" in
>             x86_64)
>               AC_CHECK_LIB(eay64,OPENSSL_config)
> @@ -1198,7 +1195,7 @@ if test "x$found_LIBMD" != "xyes"; then
>   fi
>   
>   case "$host_os" in
> -  *mingw* | *cygwin* | *msys*)
> +  *mingw* | *msys*)
>   	CRYPTO_CHECK_WIN(MD5, CALG_MD5)
>   	CRYPTO_CHECK_WIN(SHA1, CALG_SHA1)
>   	CRYPTO_CHECK_WIN(SHA256, CALG_SHA_256)
> --8<---------------cut here---------------end--------------->8---
> 
> With these, the package builds correctly.  I have a few test fails that
> all look like ATWIL / permission problems to me and none look like they
> weren't present before.
> 
> If you'd rather have me co-maintain or adopt the package, let me know.

You are the new maintainer

> 
> Regards,
> Achim.
> 

Thanks
  Marco


  reply	other threads:[~2020-10-30 10:23 UTC|newest]

Thread overview: 4+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2020-10-27 10:32 Achim Gratz
2020-10-30 10:23 ` Marco Atzeri [this message]
2020-10-30 19:36   ` [ITA] libarchive / mingw64-{i686,x86_64}-libarchive Achim Gratz
2020-10-30 20:16     ` Marco Atzeri

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=b1a7d0e1-5c1d-6321-312f-519530ca1c87@gmail.com \
    --to=marco.atzeri@gmail.com \
    --cc=cygwin-apps@cygwin.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
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 read-only IMAP folder(s) and NNTP newsgroup(s).