emacs-bug-tracker
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[debbugs-tracker] bug#29386: closed (Update taglib to 1.11.1.)


From: GNU bug Tracking System
Subject: [debbugs-tracker] bug#29386: closed (Update taglib to 1.11.1.)
Date: Tue, 21 Nov 2017 21:26:01 +0000

Your message dated Tue, 21 Nov 2017 22:25:30 +0100
with message-id <address@hidden>
and subject line Re: [bug#29386] Update taglib to 1.11.1.
has caused the debbugs.gnu.org bug report #29386,
regarding Update taglib to 1.11.1.
to be marked as done.

(If you believe you have received this mail in error, please contact
address@hidden)


-- 
29386: http://debbugs.gnu.org/cgi/bugreport.cgi?bug=29386
GNU Bug Tracking System
Contact address@hidden with problems
--- Begin Message --- Subject: Update taglib to 1.11.1. Date: Tue, 21 Nov 2017 21:11:58 +0000
Hello Guix!

This is a patch to update taglib. I've had to enable building as a
shared library otherwise gst-plugins-good would fail to link. Also,
`guix lint' warned about the home page redirecting to http://taglib.org
so I thought I'd fix that too.

Finally, I tried to enable the tests by adding "-DBUILD_TESTS" as a
configure flag but they are explicitely disabled if building the shared
library:
~~~
if(BUILD_TESTS AND NOT BUILD_SHARED_LIBS)
  enable_testing()
  add_subdirectory(tests)
endif()
~~~

Oh well, how does this look?

Thanks!
Pierre

Attachment: 0001-gnu-taglib-Update-to-1.11.1-and-fix-home-page.patch
Description: Text Data


--- End Message ---
--- Begin Message --- Subject: Re: [bug#29386] Update taglib to 1.11.1. Date: Tue, 21 Nov 2017 22:25:30 +0100 User-agent: Notmuch/0.25.2 (https://notmuchmail.org) Emacs/25.3.1 (x86_64-pc-linux-gnu)
Pierre Langlois <address@hidden> writes:

> Hello Guix!
>
> This is a patch to update taglib. I've had to enable building as a
> shared library otherwise gst-plugins-good would fail to link. Also,
> `guix lint' warned about the home page redirecting to http://taglib.org
> so I thought I'd fix that too.

Hello Pierre,

Thank you for this patch, and for checking dependent packages!  I've
applied it after fixing a typo in the commit message (home page).

> Finally, I tried to enable the tests by adding "-DBUILD_TESTS" as a
> configure flag but they are explicitely disabled if building the shared
> library:
> ~~~
> if(BUILD_TESTS AND NOT BUILD_SHARED_LIBS)
>   enable_testing()
>   add_subdirectory(tests)
> endif()
> ~~~
>
> Oh well, how does this look?

OK!  Perhaps we could do a separate build just for the tests, but that
should be in a standalone patch anyway.

Pushed as d1451fa0d0b27a20ef3a1b3fe8d53a995f0c4d85.

Attachment: signature.asc
Description: PGP signature


--- End Message ---

reply via email to

[Prev in Thread] Current Thread [Next in Thread]