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

Bug#962650: libcamera: API and ABI appear to be changing without a SONAME bump



Source: libcamera
Version: 0~git20200417+35269f0-2
Severity: important
Tags: upstream

d4f81217 "New upstream version 0~git20200513+924778e" contains ABI breaks
in headers that appear to be public, for example the ones quoted below.
(I haven't looked any further back in history.)

If this library cannot be Policy §8.1-compliant because its ABI
is not yet stable, it should probably be moved to experimental, or
distributed as a static-only library for now, or (as a last resort)
given a Debian-specific SONAME.

Please increase the severity of this bug report to serious if this is
indeed an ABI break: it certainly shouldn't be going into testing with
an unstable ABI.

Thanks,
    smcv

diff --git a/include/libcamera/stream.h b/include/libcamera/stream.h
index 18142dc..b3cbea3 100644
--- a/include/libcamera/stream.h
+++ b/include/libcamera/stream.h
@@ -42,6 +42,7 @@ struct StreamConfiguration {

        PixelFormat pixelFormat;
        Size size;
+       unsigned int stride;

        unsigned int bufferCount;

diff --git a/include/ipa/ipa_interface.h b/include/ipa/ipa_interface.h
index e65844b..dc9fc71 100644
--- a/include/ipa/ipa_interface.h
+++ b/include/ipa/ipa_interface.h
@@ -63,13 +88,15 @@ struct ipa_callback_ops {
 struct ipa_context_ops {
        void (*destroy)(struct ipa_context *ctx);
        void *(*get_interface)(struct ipa_context *ctx);
-       void (*init)(struct ipa_context *ctx);
+       void (*init)(struct ipa_context *ctx,
+                    const struct ipa_settings *settings);
        int (*start)(struct ipa_context *ctx);
        void (*stop)(struct ipa_context *ctx);
        void (*register_callbacks)(struct ipa_context *ctx,
                                   const struct ipa_callback_ops *callbacks,
                                   void *cb_ctx);
        void (*configure)(struct ipa_context *ctx,
+                         const struct ipa_sensor_info *sensor_info,
                          const struct ipa_stream *streams,
                          unsigned int num_streams,
                          const struct ipa_control_info_map *maps,


Reply to: