Merge ~vpa1977/ubuntu/+source/libwebm:LP2004442-add-headers into ubuntu/+source/libwebm:ubuntu/devel

Proposed by Vladimir Petko
Status: Needs review
Proposed branch: ~vpa1977/ubuntu/+source/libwebm:LP2004442-add-headers
Merge into: ubuntu/+source/libwebm:ubuntu/devel
Diff against target: 121 lines (+87/-1)
4 files modified
debian/changelog (+7/-0)
debian/control (+2/-1)
debian/patches/0004-add-public-headers.patch (+77/-0)
debian/patches/series (+1/-0)
Reviewer Review Type Date Requested Status
Canonical Server MOTU reviewers Pending
Canonical Server Pending
git-ubuntu import Pending
Review via email: mp+436986@code.launchpad.net

Description of the change

In order to switch aom package to building with system libwebm,
additional headers need to be published.

The change adds
    - /usr/include/webm/common/webmids.h
    - /usr/include/webm/mkvparser/mkvparser.h
    - /usr/include/webm/mkvparser/mkvreader.h
    - /usr/include/webm/mkvmuxer/mkvmuxer.h
    - /usr/include/webm/mkvmuxer/mkvmuxertypes.h
    - /usr/include/webm/mkvmuxer/mkvuxerutil.h
    - /usr/include/webm/mkvmuxer/mkvmuxerutil.h

needed to build aom package.

PPA: ppa:vpa1977/libheif

Testing:
 - Install package. Headers listed above should be included:

# add-apt-repository ppa:vpa1977/libheif
...
# apt install libwebm-dev
Reading package lists... Done
Building dependency tree... Done
Reading state information... Done
The following additional packages will be installed:
  libwebm1
The following NEW packages will be installed:
  libwebm-dev libwebm1
0 upgraded, 2 newly installed, 0 to remove and 0 not upgraded.
Need to get 237 kB of archives.
After this operation, 938 kB of additional disk space will be used.
Do you want to continue? [Y/n] Y
Get:1 https://ppa.launchpadcontent.net/vpa1977/libheif/ubuntu lunar/main amd64 libwebm1 amd64 1.0.0.29-1ubuntu1~20230208-202302080121~ubuntu23.04.1 [197 kB]
Get:2 https://ppa.launchpadcontent.net/vpa1977/libheif/ubuntu lunar/main amd64 libwebm-dev amd64 1.0.0.29-1ubuntu1~20230208-202302080121~ubuntu23.04.1 [39.6 kB]
Fetched 237 kB in 3s (69.0 kB/s)
perl: warning: Setting locale failed.
perl: warning: Please check that your locale settings:
 LANGUAGE = "en_NZ:en",
 LC_ALL = (unset),
 LANG = "en_NZ.UTF-8"
    are supported and installed on your system.
perl: warning: Falling back to the standard locale ("C").
Selecting previously unselected package libwebm1:amd64.
(Reading database ... 15249 files and directories currently installed.)
Preparing to unpack .../libwebm1_1.0.0.29-1ubuntu1~20230208-202302080121~ubuntu23.04.1_amd64.deb ...
Unpacking libwebm1:amd64 (1.0.0.29-1ubuntu1~20230208-202302080121~ubuntu23.04.1) ...
Selecting previously unselected package libwebm-dev:amd64.
Preparing to unpack .../libwebm-dev_1.0.0.29-1ubuntu1~20230208-202302080121~ubuntu23.04.1_amd64.deb ...
Unpacking libwebm-dev:amd64 (1.0.0.29-1ubuntu1~20230208-202302080121~ubuntu23.04.1) ...
Setting up libwebm1:amd64 (1.0.0.29-1ubuntu1~20230208-202302080121~ubuntu23.04.1) ...
Setting up libwebm-dev:amd64 (1.0.0.29-1ubuntu1~20230208-202302080121~ubuntu23.04.1) ...
Processing triggers for libc-bin (2.36-0ubuntu4) ...

# find /usr/include/webm/
/usr/include/webm/
/usr/include/webm/callback.h
/usr/include/webm/webm_parser.h
/usr/include/webm/mkvparser
/usr/include/webm/mkvparser/mkvreader.h
/usr/include/webm/mkvparser/mkvparser.h
/usr/include/webm/dom_types.h
/usr/include/webm/istream_reader.h
/usr/include/webm/element.h
/usr/include/webm/id.h
/usr/include/webm/file_reader.h
/usr/include/webm/common
/usr/include/webm/common/webmids.h
/usr/include/webm/status.h
/usr/include/webm/mkvmuxer
/usr/include/webm/mkvmuxer/mkvmuxer.h
/usr/include/webm/mkvmuxer/mkvmuxertypes.h
/usr/include/webm/mkvmuxer/mkvmuxerutil.h
/usr/include/webm/mkvmuxer/mkvwriter.h
/usr/include/webm/buffer_reader.h
/usr/include/webm/reader.h

#

To post a comment you must log in.
Revision history for this message
Vladimir Petko (vpa1977) wrote :

Notes:
 - vcpkg exports this set of headers:
https://github.com/microsoft/vcpkg/commit/b11941140ce64dd4ccefec44fed6a7719ba3aaf5

I will check in webm mailing list if they have
any guarantees of the API/ABI compatibility.

Unmerged commits

209421d... by Vladimir Petko

changelog

c4c3b0f... by Vladimir Petko

control

a1346db... by Vladimir Petko

add additional public headers to libwebm library

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1diff --git a/debian/changelog b/debian/changelog
2index fa4019e..da37b5b 100644
3--- a/debian/changelog
4+++ b/debian/changelog
5@@ -1,3 +1,10 @@
6+libwebm (1.0.0.29-1ubuntu1) lunar; urgency=medium
7+
8+ * debian/patches/0004-add-public-headers.patch: Add additional public headers
9+ for aom build
10+
11+ -- Vladimir Petko <vladimir.petko@canonical.com> Wed, 08 Feb 2023 13:53:37 +1300
12+
13 libwebm (1.0.0.29-1) unstable; urgency=medium
14
15 * New upstream version 1.0.0.29
16diff --git a/debian/control b/debian/control
17index 98fde10..1b16049 100644
18--- a/debian/control
19+++ b/debian/control
20@@ -1,7 +1,8 @@
21 Source: libwebm
22 Priority: optional
23 Section: libs
24-Maintainer: Debian Multimedia Maintainers <debian-multimedia@lists.debian.org>
25+Maintainer: Ubuntu Developers <ubuntu-devel-discuss@lists.ubuntu.com>
26+XSBC-Original-Maintainer: Debian Multimedia Maintainers <debian-multimedia@lists.debian.org>
27 Uploaders: Vasyl Gello <vasek.gello@gmail.com>
28 Build-Depends: debhelper-compat (= 13),
29 cmake,
30diff --git a/debian/patches/0004-add-public-headers.patch b/debian/patches/0004-add-public-headers.patch
31new file mode 100644
32index 0000000..c6f33b7
33--- /dev/null
34+++ b/debian/patches/0004-add-public-headers.patch
35@@ -0,0 +1,77 @@
36+--- a/CMakeLists.txt
37++++ b/CMakeLists.txt
38+@@ -67,15 +67,23 @@
39+ "${LIBWEBM_SRC_DIR}/common/hdr_util.h"
40+ "${LIBWEBM_SRC_DIR}/common/webmids.h")
41+
42+-set(mkvmuxer_sources
43+- "${LIBWEBM_SRC_DIR}/mkvmuxer/mkvmuxer.cc"
44++set(webmid_public_headers
45++ "${LIBWEBM_SRC_DIR}/common/webmids.h"
46++)
47++set(mkvmuxer_public_headers
48+ "${LIBWEBM_SRC_DIR}/mkvmuxer/mkvmuxer.h"
49+ "${LIBWEBM_SRC_DIR}/mkvmuxer/mkvmuxertypes.h"
50+- "${LIBWEBM_SRC_DIR}/mkvmuxer/mkvmuxerutil.cc"
51+ "${LIBWEBM_SRC_DIR}/mkvmuxer/mkvmuxerutil.h"
52+- "${LIBWEBM_SRC_DIR}/mkvmuxer/mkvwriter.cc"
53+ "${LIBWEBM_SRC_DIR}/mkvmuxer/mkvwriter.h"
54+- "${LIBWEBM_SRC_DIR}/common/webmids.h")
55++)
56++
57++set(mkvmuxer_sources
58++ ${webmid_public_headers}
59++ ${mkvmuxer_public_headers}
60++ "${LIBWEBM_SRC_DIR}/mkvmuxer/mkvmuxer.cc"
61++ "${LIBWEBM_SRC_DIR}/mkvmuxer/mkvmuxerutil.cc"
62++ "${LIBWEBM_SRC_DIR}/mkvmuxer/mkvwriter.cc"
63++)
64+
65+ set(mkvmuxer_sample_sources
66+ "${LIBWEBM_SRC_DIR}/mkvmuxer_sample.cc"
67+@@ -87,12 +95,17 @@
68+ "${LIBWEBM_SRC_DIR}/testing/test_util.cc"
69+ "${LIBWEBM_SRC_DIR}/testing/test_util.h")
70+
71++set(mkvparser_public_headers
72++ "${LIBWEBM_SRC_DIR}/mkvparser/mkvreader.h"
73++ "${LIBWEBM_SRC_DIR}/mkvparser/mkvparser.h"
74++)
75++
76+ set(mkvparser_sources
77++ ${mkvparser_public_headers}
78++ ${webmid_public_headers}
79+ "${LIBWEBM_SRC_DIR}/mkvparser/mkvparser.cc"
80+- "${LIBWEBM_SRC_DIR}/mkvparser/mkvparser.h"
81+ "${LIBWEBM_SRC_DIR}/mkvparser/mkvreader.cc"
82+- "${LIBWEBM_SRC_DIR}/mkvparser/mkvreader.h"
83+- "${LIBWEBM_SRC_DIR}/common/webmids.h")
84++)
85+
86+ set(mkvparser_sample_sources "${LIBWEBM_SRC_DIR}/mkvparser_sample.cc")
87+ set(mkvparser_tests_sources
88+@@ -323,7 +336,24 @@
89+
90+ # Targets.
91+ add_library(mkvmuxer OBJECT ${mkvmuxer_sources})
92++set_target_properties(mkvmuxer PROPERTIES PUBLIC_HEADER
93++ "${mkvmuxer_public_headers}")
94++install(TARGETS mkvmuxer
95++ PUBLIC_HEADER DESTINATION ${CMAKE_INSTALL_INCLUDEDIR}/webm/mkvmuxer)
96++
97++add_library(webmid_common INTERFACE)
98++set_target_properties(webmid_common PROPERTIES PUBLIC_HEADER
99++ "${webmid_public_headers}")
100++install(TARGETS webmid_common
101++ PUBLIC_HEADER DESTINATION ${CMAKE_INSTALL_INCLUDEDIR}/webm/common)
102++
103++
104+ add_library(mkvparser OBJECT ${mkvparser_sources})
105++set_target_properties(mkvparser PROPERTIES PUBLIC_HEADER
106++ "${mkvparser_public_headers}")
107++install(TARGETS mkvparser
108++ PUBLIC_HEADER DESTINATION ${CMAKE_INSTALL_INCLUDEDIR}/webm/mkvparser)
109++
110+ add_library(webvtt_common OBJECT ${webvtt_common_sources})
111+
112+ add_library(webm ${libwebm_common_sources}
113diff --git a/debian/patches/series b/debian/patches/series
114index a6e1979..e251fca 100644
115--- a/debian/patches/series
116+++ b/debian/patches/series
117@@ -1,3 +1,4 @@
118 0001-Use-system-googletest.patch
119 0002-Build-install-libwebm-tools.patch
120 0003-Provide-SOVERSION.patch
121+0004-add-public-headers.patch

Subscribers

People subscribed via source and target branches