Hi,
attached a patch that adds a pkg-config file in case a shared library is
built and installed. Note that the version will be needed for the
pkg-config file. Would it make sense to have pkg-config files for each
library individually in addition to the combining one?
--
Benjamin Drung
Debian
space there.
> Also, I'm not going to "fix what's not broken".
Thanks. I have attached a smaller, less intrusive patch for the trailing
space removal. These removals have no bad side effect.
--
Benjamin Drung
Debian & Ubuntu Developer
diff --git a/BasicUsageEnv
a variable. strip-trailing-spaces.patch is attached.
--
Benjamin Drung
Debian & Ubuntu Developer
Description: Pass LDFLAGS to the shared library linker
Author: Benjamin Drung
--- a/config.linux-with-shared-libraries
+++ b/config.linux-with-shared-libraries
@@ -38,7 +38,7 @@
LIBRARY_LINK =
PREFIX=/usr" still works (I have tested it).
Can you please leave LIBDIR = $(PREFIX)/lib instead of hard-coding the
libdir? If a user sets PREFIX to /usr, the default libdir should
be /usr/lib instead of /usr/local/lib.
--
Benjamin Drung
Debian & Ubuntu Developer
___
Am Montag, den 24.12.2012, 10:36 +1000 schrieb Ross Finlayson:
>
> On Dec 23, 2012, at 12:03 AM, Benjamin Drung
> wrote:
>
> > Am Samstag, den 22.12.2012, 13:22 +1000 schrieb Ross Finlayson:
> > > 2/ The second 'patch' seems wrong to me. I don't u
erstood that if a developer wants to enable these pieces of code,
> then they need to build the library themselves - with appropriate
> compile-time flags - rather than rely upon a pre-built library. This
> is no different. So the third patch should be removed.
I have removed the patch. Please keep in mind that future changes like
that causes an ABI breakage (and therefore a soname bump).
--
Benjamin Drung
Debian & Ubuntu Developer
___
live-devel mailing list
live-devel@lists.live555.com
http://lists.live555.com/mailman/listinfo/live-devel
Am Sonntag, den 23.12.2012, 16:40 +1000 schrieb Ross Finlayson:
> > Spaces are used instead of the required tabs in the install target
>
> Oops - fixed now. (I understand why many people hate Makefiles :-)
It's still not fixed in 2012.12.23 and I actually like Makefiles.
-
h:260:8: error: previous definition of ‘struct
ip_mreq_source’
--
Benjamin Drung
Debian & Ubuntu Developer
___
live-devel mailing list
live-devel@lists.live555.com
http://lists.live555.com/mailman/listinfo/live-devel
gt; presume) anywhere else in the Debian distribution either.
I see a problem here: We build a static/shared library without the
deprecated old, synchronous RTSPClient interface. If a user defines
RTSPCLIENT_SYNCHRONOUS_INTERFACE, he will get the old header functions,
but linking with the s
; > VLC
> > against the shared library version. Therefore I am not sure if these
> > dpkg-shlibdeps warning are a problem. It's better to diagnose the
> > cause
> > instead of ignoring a warning.
>
>
> I'll leave it to someone who understan
Hi,
please add a distclean target which removes all files that are not part
of the source tarball. distclean would depend on clean and additional
remove all Makefiles. A patch that does this is attached.
--
Benjamin Drung
Debian & Ubuntu Developer
diff --git a/Makefile.tail b/Makefile.
Hi,
we carry three patches (attached) in the Debian package that predate my
involvement in the Debian packaging. Are these patches acceptable?
--
Benjamin Drung
Debian & Ubuntu Developer
---
liveMedia/H263plusVideoRTPSink.cpp |2 +-
liveMedia/MPEG1or2Demux.cpp|4 ++--
2 f
s are linked against the applications??)
These warning are generated by dpkg-shlibdeps. I was able to build VLC
against the shared library version. Therefore I am not sure if these
dpkg-shlibdeps warning are a problem. It's better to diagnose the cause
instead of ignoring a warning.
--
Ben
Hi,
please put the changelog.txt [1] in the source tarball. We like to put
this changelog file into our Debian package and it will make it simpler
if the changelog file is in the source tarball.
[1] http://live555.com/liveMedia/public/changelog.txt
--
Benjamin Drung
Debian & Ubuntu Devel
Hi,
please support CPPFLAGS, CFLAGS, CXXFLAGS, LDFLAGS. It would be nice if
the build system honors these variables if they are set by the user. In
Debian, hardening flags are passed through these variables. A patch that
add the support is attached.
--
Benjamin Drung
Debian & Ubuntu Devel
.
They should be used on other systems (e.g. BSD) too.
3) Some symbols used by the shared libraries are not found in none of
the libraries (build.log attached).
--
Benjamin Drung
Debian & Ubuntu Developer
Description: Add an install target to the build system.
Author: Benjamin Drung
---
changes, or is removed (i.e., so that
> existing code that uses the library would likely be incompatible with
> the new version).
Increment current, reset others: c+1:0:0
> 3/ One or more interfaces were added, but no existing interfaces were
> changed or removed (i.e., so that existing code th
any interfaces have been added since the last public release, then
increment
# age.
# 6. If any interfaces have been removed since the last public release, then
set age
# to 0.
SHARED_VERSION_INFO="0:0:0"
[1] See 010_propagate_cflags.diff and 301_hardening.patch on
http://patc
shared library by taking care of a
proper SONAME version?
[1] http://bugs.debian.org/662774
--
Benjamin Drung
Debian & Ubuntu Developer
___
live-devel mailing list
live-devel@lists.live555.com
http://lists.live555.com/mailman/listinfo/live-devel
19 matches
Mail list logo
