[ITA] libexif-0.6.22-1

classic Classic list List threaded Threaded
14 messages Options
Reply | Threaded
Open this post in threaded view
|

[ITA] libexif-0.6.22-1

cygwin-apps mailing list
Hi!

ITA for libexif, which has been maintained by Yaakov [1].

Now, the latest upstream of libexif is 0.6.22 [2].

A new candidate cygport file is placed at [3].
But it is not tested on Cygwin AppVeyor CI, yet.

Generated package files are placed at [4] and [5]:

* libexif-common-0.6.22-1.tar.xz
* libexif-devel-0.6.22-1.tar.xz
* libexif12-0.6.22-1.tar.xz

* libexif-0.6.22-1-src.tar.xz
* libexif-debuginfo-0.6.22-1.tar.xz

[1]: https://cygwin.com/git/?p=git/cygwin-packages/libexif.git
[2]: https://github.com/libexif/libexif/releases
[3]: https://github.com/cygwin-lem/libexif-cygport/tree/n_0.6.22-1
[4]: https://cygwin-lem.github.io/libexif-cygport/
[5]: https://github.com/cygwin-lem/libexif-cygport/tree/n_0.6.22-1_gh-pages


Regards,

Lem

Reply | Threaded
Open this post in threaded view
|

Re: [ITA] libexif-0.6.22-1

cygwin-apps mailing list
On Thu, 31 Dec 2020 08:33:50 +0900, Lemures Lemniscati

> Hi!
>
> ITA for libexif, which has been maintained by Yaakov [1].
>
> Now, the latest upstream of libexif is 0.6.22 [2].
>
> A new candidate cygport file is placed at [3].
> But it is not tested on Cygwin AppVeyor CI, yet.
>
> Generated package files are placed at [4] and [5]:
>
> * libexif-common-0.6.22-1.tar.xz
> * libexif-devel-0.6.22-1.tar.xz
> * libexif12-0.6.22-1.tar.xz
>
> * libexif-0.6.22-1-src.tar.xz
> * libexif-debuginfo-0.6.22-1.tar.xz
>
> [1]: https://cygwin.com/git/?p=git/cygwin-packages/libexif.git
> [2]: https://github.com/libexif/libexif/releases
> [3]: https://github.com/cygwin-lem/libexif-cygport/tree/n_0.6.22-1
> [4]: https://cygwin-lem.github.io/libexif-cygport/
> [5]: https://github.com/cygwin-lem/libexif-cygport/tree/n_0.6.22-1_gh-pages

Sorry, this is premature.
I'll do ITA again, later.

Regards,

Lem

Reply | Threaded
Open this post in threaded view
|

Re: [ITA] libexif-0.6.22-1

cygwin-apps mailing list
Hi,


I'm trying to build libexif and exif, and I need help.

exif is troublesome:

  i686 build failed on my local machine,
  while it's successfully built on AppVeyor CI.

(1) It's successfully built on AppVeryor CI for both of x86_64 and i686:
    ( ID 1550: https://ci.appveyor.com/project/cygwin/scallywag/builds/37055047 )
    and on my local machine.

(2) For x86_64, it's built successfully on my local machine,
    but, for i686, it's failed on local machine.

I've reinstalled cygwin32 and tried again, but the results are same.


The cygport file and a related patch is here:
  https://cygwin.com/git/?p=git/cygwin-packages/playground.git;a=tree;h=d5d2112d655cb0f963792298e4fbca7f6732b4de;hb=ae08a609889d3ee1397d3bd7fd9817bdb33fc1c6
  or https://github.com/cygwin-lem/exif-cygport/tree/n_0.6.22-1

compile.log for i686 on my machine, and /etc/setup/intalled.db
are attached to this mail.


> libtool:   error: cannot find the library '/usr/lib/libintl.la' or unhandled argument '/usr/lib/libintl.la'

It seems that libtool might go wrong.
But I don't know how to resolve it.



Regards,

Lem


P.S.
  libexif is now prepared, as AppVeyor CI has successfully built it
  ( ID 1549: https://ci.appveyor.com/project/cygwin/scallywag/builds/37054930 ),
  and on my local machine.

exif-0.6.22-1-compile.log (58K) Download Attachment
installed.db (10K) Download Attachment
Reply | Threaded
Open this post in threaded view
|

Re: [ITA] libexif-0.6.22-1

Achim Gratz
Lemures Lemniscati via Cygwin-apps writes:
> exif is troublesome:
>
>   i686 build failed on my local machine,
>   while it's successfully built on AppVeyor CI.

If it's not due to fork errors, then I'd suspect some difference in the
way Windows itself is set up.

> I've reinstalled cygwin32 and tried again, but the results are same.

A reinstallation is unlikely to do anything useful unless you suspect
that you've messed it up previously.  You can trigger a full rebase once
in a while, although I haven't needed that in a long time.

My recommendation is still to keep the (32bit and 64bit) Cygwin
installations that you use for packaging separate from the one you do
any daily work in and keep them in a second-level directory of a
physical drive (like D:\Freeware\Cygwin32).  Do not use any /cygdrive
resources, instead mount everything you need via /etc/fstab.d/builduser.
Keep any locale settings as plain as possible.

>   CCLD     exif.exe
> libtool:   error: cannot find the library '/usr/lib/libintl.la' or unhandled argument '/usr/lib/libintl.la'

Next time you might want to run that again with "V=1" or whatever
invocation removes the cloaking of the linker command line and see if
that gives you a better clue as to what is the problem here.

So it turns out to fail the same on my system locally… and if you look
into .appveyor.yaml you'll find this little nugget:

--8<---------------cut here---------------start------------->8---
# Work around a libpopt-devel-1.16-2 packaging issue that causes the error:
#   libtool: error: cannot find the library '/usr/lib/libintl.la' or unhandled argument '/usr/lib/libintl.la'
# See http://www.cygwin.com/ml/cygwin-apps/2017-07/msg00005.html
- cmd: if [%CONFIG%]==[cygwin] (
    C:\%CONFIG%\bin\bash -e -l -c "rm /usr/lib/libpopt.la" )
--8<---------------cut here---------------end--------------->8---

…and indeed that makes things work.  :-)


Regards,
Achim.
--
+<[Q+ Matrix-12 WAVE#46+305 Neuron microQkb Andromeda XTk Blofeld]>+

Samples for the Waldorf Blofeld:
http://Synth.Stromeko.net/Downloads.html#BlofeldSamplesExtra
Reply | Threaded
Open this post in threaded view
|

Re: [ITA] libexif-0.6.22-1

Achim Gratz
Achim Gratz writes:
> # Work around a libpopt-devel-1.16-2 packaging issue that causes the error:
> #   libtool: error: cannot find the library '/usr/lib/libintl.la' or unhandled argument '/usr/lib/libintl.la'
> # See http://www.cygwin.com/ml/cygwin-apps/2017-07/msg00005.html
> - cmd: if [%CONFIG%]==[cygwin] (
>     C:\%CONFIG%\bin\bash -e -l -c "rm /usr/lib/libpopt.la" )
>
> …and indeed that makes things work.  :-)

This is the reason it works on the CI:

curl -fsSL https://gist.githubusercontent.com/jon-turney/0338af595313f598bfab15a0ac0df847/raw/bd0eeca6be899e7846aa988fbcf15e4e12f5f842/zp_libtool_cleanlafiles.sh -o %CYGWIN_ROOT%\etc\postinstall\zp_libtool_cleanlafiles.sh

--8<---------------cut here---------------start------------->8---
rmla=()
for la in $(find /usr/lib/ -maxdepth 1 -name '*.la')
do
        # modules in /usr/lib (e.g. aRts, guile) are generally libltdl-based
        if grep -q '^shouldnotlink=yes$' $la
        then
                continue
        fi
        case ${la##*/} in
        # ltdl.m4 requires this to be present
        libltdl.la) continue ;;
        # gnucash relies on .la loading due to the 'cyg' prefix
        libgnc-*.la) continue ;;
        esac

        rmla+=($la)
done

rm -f ${rmla[*]}
--8<---------------cut here---------------end--------------->8---



Regards,
Achim.
--
+<[Q+ Matrix-12 WAVE#46+305 Neuron microQkb Andromeda XTk Blofeld]>+

Factory and User Sound Singles for Waldorf Blofeld:
http://Synth.Stromeko.net/Downloads.html#WaldorfSounds
Reply | Threaded
Open this post in threaded view
|

Re: [ITA] libexif-0.6.22-1

cygwin-apps mailing list
On Thu, 31 Dec 2020 09:49:33 +0100, Achim Gratz

> Achim Gratz writes:
> > # Work around a libpopt-devel-1.16-2 packaging issue that causes the error:
> > #   libtool: error: cannot find the library '/usr/lib/libintl.la' or unhandled argument '/usr/lib/libintl.la'
> > # See http://www.cygwin.com/ml/cygwin-apps/2017-07/msg00005.html
> > - cmd: if [%CONFIG%]==[cygwin] (
> >     C:\%CONFIG%\bin\bash -e -l -c "rm /usr/lib/libpopt.la" )
> >
> > …and indeed that makes things work.  :-)
>
> This is the reason it works on the CI:
>
> curl -fsSL https://gist.githubusercontent.com/jon-turney/0338af595313f598bfab15a0ac0df847/raw/bd0eeca6be899e7846aa988fbcf15e4e12f5f842/zp_libtool_cleanlafiles.sh -o %CYGWIN_ROOT%\etc\postinstall\zp_libtool_cleanlafiles.sh
>
> --8<---------------cut here---------------start------------->8---
> rmla=()
> for la in $(find /usr/lib/ -maxdepth 1 -name '*.la')
> do
>         # modules in /usr/lib (e.g. aRts, guile) are generally libltdl-based
>         if grep -q '^shouldnotlink=yes$' $la
>         then
>                 continue
>         fi
>         case ${la##*/} in
>         # ltdl.m4 requires this to be present
>         libltdl.la) continue ;;
>         # gnucash relies on .la loading due to the 'cyg' prefix
>         libgnc-*.la) continue ;;
>         esac
>
>         rmla+=($la)
> done
>
> rm -f ${rmla[*]}
> --8<---------------cut here---------------end--------------->8---
>
>
>
> Regards,
> Achim.

Thank you very much!
It works like a magic.


libpopt-devel-1.16-2 for x86 has /usr/lib/libpopt.la

$ uname -srm; cygcheck -c libpopt-devel; cygcheck -l libpopt-devel
CYGWIN_NT-10.0-WOW 3.1.7(0.340/5/3) i686
Cygwin Package Information
Package              Version        Status
libpopt-devel        1.16-2         Incomplete
/usr/include/popt.h
/usr/lib/libpopt.dll.a
/usr/lib/libpopt.la
/usr/lib/pkgconfig/popt.pc
/usr/share/man/man3/popt.3.gz

Regards,

Lem


P.S. By the way, I'm also planning to package popt :)

Reply | Threaded
Open this post in threaded view
|

Re: [ITA] libexif-0.6.22-1

cygwin-apps mailing list
Helped by Achim Gratz, now I can tell ITA for libexif-0.6.22-1 again,
and, in a following mail, ITA for exif-0.6.22-1.

--
ITA for libexif, which has been maintained by Yaakov [1].

Now, the latest upstream of libexif is 0.6.22 [2].

A new candidate cygport file is placed at [3].
And, it's been tested on Cygwin AppVeyor CI (ID 1557: [4]).

Generated package files are placed at [5] and [6]:

* libexif-common-0.6.22-1.tar.xz
* libexif-devel-0.6.22-1.tar.xz
* libexif12-0.6.22-1.tar.xz

* libexif-0.6.22-1-src.tar.xz
* libexif-debuginfo-0.6.22-1.tar.xz

[1]: https://cygwin.com/git/?p=git/cygwin-packages/libexif.git
[2]: https://github.com/libexif/libexif/releases
[3]: https://github.com/cygwin-lem/libexif-cygport/tree/n_0.6.22-1
[4]: https://ci.appveyor.com/project/cygwin/scallywag/builds/37058262
[5]: https://cygwin-lem.github.io/libexif-cygport/
[6]: https://github.com/cygwin-lem/libexif-cygport/tree/n_0.6.22-1_gh-pages


Regards,

Lem

Reply | Threaded
Open this post in threaded view
|

Re: [ITA] libexif-0.6.22-1

cygwin-apps mailing list
On 31.12.2020 11:54, Lemures Lemniscati via Cygwin-apps wrote:

> Helped by Achim Gratz, now I can tell ITA for libexif-0.6.22-1 again,
> and, in a following mail, ITA for exif-0.6.22-1.
>
> --
> ITA for libexif, which has been maintained by Yaakov [1].
>
> Now, the latest upstream of libexif is 0.6.22 [2].
>
> A new candidate cygport file is placed at [3].
> And, it's been tested on Cygwin AppVeyor CI (ID 1557: [4]).
>
> Generated package files are placed at [5] and [6]:
>
> * libexif-common-0.6.22-1.tar.xz
> * libexif-devel-0.6.22-1.tar.xz
> * libexif12-0.6.22-1.tar.xz
>
> * libexif-0.6.22-1-src.tar.xz
> * libexif-debuginfo-0.6.22-1.tar.xz

it builds fine and pass the tests

I would remove the pull for the shared lib

printf -v "libexif${ABI}_REQUIRES" "%s" "libexif-common"

of the general doc

 >>> libexif-common-0.6.22-1.tar.xz
usr/share/doc/
usr/share/doc/libexif/
usr/share/doc/libexif/ABOUT-NLS
usr/share/doc/libexif/AUTHORS
usr/share/doc/libexif/ChangeLog
usr/share/doc/libexif/COPYING
usr/share/doc/libexif/NEWS
usr/share/doc/libexif/README
usr/share/doc/libexif/SECURITY.md

but it is just my opinion

> Regards,
>
> Lem
>

GTG

Thanks & Regards
Marco
Reply | Threaded
Open this post in threaded view
|

Re: [ITA] libexif-0.6.22-1

cygwin-apps mailing list
On Thu, 31 Dec 2020 16:44:20 +0100, Marco Atzeri via Cygwin-apps

> On 31.12.2020 11:54, Lemures Lemniscati via Cygwin-apps wrote:
> > Helped by Achim Gratz, now I can tell ITA for libexif-0.6.22-1 again,
> > and, in a following mail, ITA for exif-0.6.22-1.
> >
> > --
> > ITA for libexif, which has been maintained by Yaakov [1].
> >
> > Now, the latest upstream of libexif is 0.6.22 [2].
> >
> > A new candidate cygport file is placed at [3].
> > And, it's been tested on Cygwin AppVeyor CI (ID 1557: [4]).
> >
> > Generated package files are placed at [5] and [6]:
> >
> > * libexif-common-0.6.22-1.tar.xz
> > * libexif-devel-0.6.22-1.tar.xz
> > * libexif12-0.6.22-1.tar.xz
> >
> > * libexif-0.6.22-1-src.tar.xz
> > * libexif-debuginfo-0.6.22-1.tar.xz
>
> it builds fine and pass the tests
>
> I would remove the pull for the shared lib
>
> printf -v "libexif${ABI}_REQUIRES" "%s" "libexif-common"
>
> of the general doc
>
>  >>> libexif-common-0.6.22-1.tar.xz
> usr/share/doc/
> usr/share/doc/libexif/
> usr/share/doc/libexif/ABOUT-NLS
> usr/share/doc/libexif/AUTHORS
> usr/share/doc/libexif/ChangeLog
> usr/share/doc/libexif/COPYING
> usr/share/doc/libexif/NEWS
> usr/share/doc/libexif/README
> usr/share/doc/libexif/SECURITY.md
>
> but it is just my opinion
>
Thank you, Marco.

I agree. And I'll remove the dependency and rebuild.

Regards,

Lem

Reply | Threaded
Open this post in threaded view
|

Re: [ITA] libexif-0.6.22-1

Achim Gratz
Lemures Lemniscati via Cygwin-apps writes:
> I agree. And I'll remove the dependency and rebuild.

Minor nit: IMHO it would have been all-around cleaner if you obsoleted
libexif-common and replaced it with libexif-doc.


Regards,
Achim.
--
+<[Q+ Matrix-12 WAVE#46+305 Neuron microQkb Andromeda XTk Blofeld]>+

Factory and User Sound Singles for Waldorf Q+, Q and microQ:
http://Synth.Stromeko.net/Downloads.html#WaldorfSounds
Reply | Threaded
Open this post in threaded view
|

Re: [ITA] libexif-0.6.22-1

cygwin-apps mailing list
On Sat, 02 Jan 2021 23:13:45 +0100, Achim Gratz
> Lemures Lemniscati via Cygwin-apps writes:
> > I agree. And I'll remove the dependency and rebuild.
>
> Minor nit: IMHO it would have been all-around cleaner if you obsoleted
> libexif-common and replaced it with libexif-doc.
>


I guess if we make libexif-doc package, it would be better to move html
document files (104 files), also, from libexiv-devel into libexif-doc.


So an expected libexiv-doc will contain

/usr/share/doc/libexif/ABOUT-NLS
/usr/share/doc/libexif/AUTHORS
/usr/share/doc/libexif/ChangeLog
/usr/share/doc/libexif/COPYING
/usr/share/doc/libexif/NEWS
/usr/share/doc/libexif/README
/usr/share/doc/libexif/SECURITY.md

and

/usr/share/doc/libexif/libexif-api.html/*

(104 files).

How about this rearrangement?

Lem

Reply | Threaded
Open this post in threaded view
|

Re: [ITA] libexif-0.6.22-1

cygwin-apps mailing list


On 03.01.2021 03:46, Lemures Lemniscati via Cygwin-apps wrote:

> On Sat, 02 Jan 2021 23:13:45 +0100, Achim Gratz
>> Lemures Lemniscati via Cygwin-apps writes:
>>> I agree. And I'll remove the dependency and rebuild.
>>
>> Minor nit: IMHO it would have been all-around cleaner if you obsoleted
>> libexif-common and replaced it with libexif-doc.
>>
>
>
> I guess if we make libexif-doc package, it would be better to move html
> document files (104 files), also, from libexiv-devel into libexif-doc.

>
> How about this rearrangement?

it makes sense

> Lem

Regards
Marco
Reply | Threaded
Open this post in threaded view
|

Re: [ITA] libexif-0.6.22-1

Achim Gratz
In reply to this post by cygwin-apps mailing list
Lemures Lemniscati via Cygwin-apps writes:
> So an expected libexiv-doc will contain
[…]
> How about this rearrangement?

That sounds OK to me.  You don't need to do another update just for that
change of course.


Regards,
Achim.
--
+<[Q+ Matrix-12 WAVE#46+305 Neuron microQkb Andromeda XTk Blofeld]>+

SD adaptations for Waldorf Q V3.00R3 and Q+ V3.54R2:
http://Synth.Stromeko.net/Downloads.html#WaldorfSDada
Reply | Threaded
Open this post in threaded view
|

Re: [ITA] libexif-0.6.22-1

cygwin-apps mailing list
On Sun, 03 Jan 2021 10:51:30 +0100, Achim Gratz
> Lemures Lemniscati via Cygwin-apps writes:
> > So an expected libexiv-doc will contain
> […]
> > How about this rearrangement?
>
> That sounds OK to me.  You don't need to do another update just for that
> change of course.

Thank you.

I've just prepared it [1]. The change will be reflected in a next update
(latest upstream or security).

[1]: https://github.com/cygwin-lem/libexif-cygport/blob/w_rearrange/libexif.cygport

Regards,

Lem