| Commit message (Collapse) | Author | Age | Files | Lines |
| |
|
|\
| |
| |
| |
| | |
Remove Autotools
See merge request GNOME/libmediaart!12
|
| |
| |
| |
| | |
In other words: meson setup, meson compile, and meson test.
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
- Use the ignore_headers argument instead of passing it to gtkdoc-scan
ourselves
- Ignore the symbol decorator for the public API
- Specify the default include
- Hide the media_art_process_get_type() function
- Build the types file, so that gtk-doc will recognise GObject types
- Add gtkdoc-check to the test suite, so that undocumented API slips
through
|
| |
| |
| |
| | |
It's pointless to document them.
|
| |
| |
| |
| | |
The description is missing.
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
- look for sources in both source and build directories
- depend on the built library
- ignore the generated, uninstalled marshallers header
- make gtk-doc track the overview.xml file
- add the missing indices
- add the missing symbols
- use a boolean `gtk_doc` build option, like every other
GNOME project
|
| |
| |
| |
| | |
There's no other documentation, and there is only one API reference.
|
| |
| |
| |
| | |
The pkg-config file is generated by Meson.
|
| |
| |
| |
| | |
The authors are available in the Git commit log.
|
|/
|
|
|
| |
There is no point in keeping two build systems in parallel, especially
when one of them is Autotools.
|
|\
| |
| |
| |
| | |
Drop Qt4 support
See merge request GNOME/libmediaart!11
|
|/
|
|
| |
Qt 4 was end-of-lifed in December 2015.
|
|\
| |
| |
| |
| | |
Don't use a field before setting it
See merge request GNOME/libmediaart!10
|
| |
| |
| |
| |
| |
| | |
The g_memdup() call was deprecated in GLib 2.68 because it used an
unsigned integer for the buffer size argument. The g_memdup2() function,
which uses a gsize, was introduced in the same version.
|
| |
| |
| |
| |
| |
| | |
We are using the data->len field in our g_memdup() call before we're
actually setting it; since the data structure is zero-filled, it means
we're not copying the buffer at all.
|
|\ \
| |/
|/|
| |
| | |
build: Export the pkg-config file in the GIR
See merge request GNOME/libmediaart!9
|
| |
| |
| |
| |
| |
| |
| |
| | |
Instead of hand-rolling our own.
Remove the bit about the C compiler: Meson prints out the compiler in
its default output. Similarly, remove the bit about the source
directory: Meson does that automatically.
|
| |
| |
| |
| | |
Something not ancient.
|
|/
|
|
| |
Tools that parse the GIR need the exported package.
|
|\
| |
| |
| |
| | |
Mandate that inputs are valid UTF8
See merge request GNOME/libmediaart!7
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
This restriction was already in place, since we passed the inputs
directly to GLib functions that expect valid UTF8.
If this library were widely used then we might proactively validate
inputs to protect against exploits. At time of writing, this module is
only used in 3 places (grilo-plugins, gnome-music and rygel), so just
check they are doing the right thing when calling the API.
See https://gitlab.gnome.org/GNOME/libmediaart/-/merge_requests/5 for
background.
|
|/
|
|
|
|
|
|
|
|
|
| |
We return NULL if input is NULL and a newly allocated empty string if
input is "". Some comments disagreed with this, and it's possible the
change causes a memory leak in some app, but the alternative of returning
NULL when passed "" is dangerous as some code may free the return value
in this case. (In fact, libmediaart itself does so).
Also, make behaviour occur independently of whether
`G_ENABLE_CONSISTENCY_CHECKS` was defined at build time.
|
|\
| |
| |
| |
| | |
ci: Update for changes in tracker-oci-images
See merge request GNOME/libmediaart!8
|
|/
|
|
|
|
|
|
|
| |
Since https://gitlab.gnome.org/GNOME/tracker-oci-images/-/merge_requests/28
the container images set 'USER tracker', so privileged ops like
installing packages are no longer possible during testcases.
So, gdk-pixbuf is added to the base images in
https://gitlab.gnome.org/GNOME/tracker-oci-images/-/merge_requests/38.
|
|\
| |
| |
| |
| | |
Initial CI config
See merge request GNOME/libmediaart!6
|
|/ |
|
|\
| |
| |
| |
| | |
Fix Qt5 backend crashing when running in Qt application
See merge request GNOME/libmediaart!2
|
| |
| |
| |
| |
| | |
Creating a dummy QCoreApplication instance is bad in itself,
but even worse when running in an app that already has one.
|
|\ \
| | |
| | |
| | |
| | | |
meson.build: don't require C++
See merge request GNOME/libmediaart!3
|
| |/
| |
| |
| |
| |
| | |
Does not mandate C++ as it is only required for extractqt.cpp
Signed-off-by: Fabrice Fontaine <fontaine.fabrice@gmail.com>
|
|\ \
| |/
|/|
| |
| | |
Remove storage class
See merge request GNOME/libmediaart!4
|
| |
| |
| |
| |
| |
| | |
After the removal of mediaartlocal, this is useless now.
It can also cause some lock-ups if gvfs is missing
|
|/ |
|
| |
|
|
|
|
|
|
|
| |
The meson build did not add the extractdummy.c to the sources, which
contains introspection annotations.
https://bugzilla.gnome.org/show_bug.cgi?id=792272
|
| |
|
|
|
|
|
| |
This was making the functions defined in extractgeneric.h not really
exported.
|
|
|
|
| |
This must be installed by the meson install as well...
|
|
|
|
| |
Allows disabling the API reference docs.
|
| |
|
|
|
|
| |
https://bugzilla.gnome.org/show_bug.cgi?id=786067
|
|
|
|
|
| |
The header is referenced by publicly installed headers, which means
installations of libmediaart without it are now broken.
|
| |
|
| |
|
| |
|
| |
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
These are hopefully complete already. I have compared an Autotools-built
and a Meson-built install of libmediaart and found only the following
differences:
* libmediaart-2.0.la isn't generated by Meson
* External references in the gtk-doc documentation are relative with
Meson and absolute with Autotools
* Some changes in generated .vapi file and .pc file
https://bugzilla.gnome.org/show_bug.cgi?id=783562
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Previously we relied on libtool's -export-symbols-regex feature, but we
are hoping to drop GNU Autotools and GNU Libtool in soon in favour of
Meson.
Meson doesn't have an equivalent feature, instead the advice is to
control symbol visibility at compile time.
The approach taken in this patch is based on Pango's build system. Pango
tells the compiler to hide symbols by default (if possible), and then defines
a _PANGO_EXTERN variable at compile time which marks a single symbol as
public. In Pango's case there is then further machinary to hide symbols
based on deprecation policies but I have not copied that here, instead I
used _LIBMEDIAART_EXTERN directly.
If a compiler doesn't support hiding symbols then the library we build
makes all symbols available, which is exactly what would happen before
on platforms where the libtool didn't have an implementation for
-export-symbols-regex.
See also:
http://mesonbuild.com/FAQ.html#how-do-i-do-the-equivalent-of-libtools-exportsymbol-and-exportregex
https://git.gnome.org/browse/pango/
https://git.gnome.org/browse/pango/tree/pango/pango-version-macros.h
https://gcc.gnu.org/wiki/Visibility
https://gcc.gnu.org/onlinedocs/gcc/Code-Gen-Options.html#Code-Gen-Options (-fvisibility)
https://bugzilla.gnome.org/show_bug.cgi?id=783562
|
|
|
|
|
|
|
|
|
|
| |
This flag was just enabling a codepath that sets QT_QPA_PLATFORM=minimal
in the environment before constructing a QCoreApplication instance.
This code path has no effect anyway: QT_QPA_PLATFORM only has an effect
if you are using QGuiApplication.
https://bugzilla.gnome.org/show_bug.cgi?id=783562
|