From: Gregor Jasny <gjasny@googlemail.com>
To: Konke Radlow <kradlow@cisco.com>
Cc: linux-media@vger.kernel.org
Subject: Re: [RFC PATCH 0/2] Add support for RDS decoding
Date: Sun, 29 Jul 2012 19:07:25 +0200 [thread overview]
Message-ID: <50156DCD.5070205@googlemail.com> (raw)
In-Reply-To: <201207271427.40172.kradlow@cisco.com>
Hello Konke,
On 7/27/12 4:27 PM, Konke Radlow wrote:
> changing the condition in the library header from
>> #if __GNUC__ >= 4
>> #define LIBV4L_PUBLIC __attribute__ ((visibility("default")))
>> #else
>> #define LIBV4L_PUBLIC
>> #endif
>
> to
>> #if HAVE_VISIBILITY
>> #define LIBV4L_PUBLIC __attribute__ ((visibility("default")))
>> #else
>> #define LIBV4L_PUBLIC
>> #endif
>
> causes linker problems for me. The public library functions can no longer be
> found. I cannot figure out why it's working for programs using libv4l2.la but
> not for programs using libv4l2rds.la
You need to include <config.h> before including this file in the utility
and library to get the HAVE_VISIBILITY definition activated.
The other option would be switching from defining HAVE_VISIBILITY in
config.h to a command line define.
Thanks,
Gregor
next prev parent reply other threads:[~2012-07-29 17:07 UTC|newest]
Thread overview: 18+ messages / expand[flat|nested] mbox.gz Atom feed top
2012-07-25 17:43 [RFC PATCH 0/2] Add support for RDS decoding Konke Radlow
2012-07-25 17:44 ` [RFC PATCH 1/2] Initial version of the RDS-decoder library Signed-off-by: Konke Radlow <kradlow@cisco.com> Konke Radlow
2012-07-25 17:44 ` [RFC PATCH 2/2] Initial version of RDS Control utility " Konke Radlow
2012-07-26 19:02 ` Gregor Jasny
2012-07-26 19:13 ` Gregor Jasny
2012-07-26 19:46 ` Konke Radlow
2012-07-27 6:50 ` Hans Verkuil
2012-07-28 11:11 ` Hans de Goede
2012-07-30 6:50 ` Hans Verkuil
2012-07-26 14:28 ` [RFC PATCH 1/2] Initial version of the RDS-decoder library " Ezequiel Garcia
2012-07-26 14:39 ` Hans Verkuil
2012-07-26 14:41 ` Ezequiel Garcia
2012-07-26 18:46 ` Gregor Jasny
2012-07-26 18:49 ` Gregor Jasny
2012-07-26 18:41 ` [RFC PATCH 0/2] Add support for RDS decoding Gregor Jasny
2012-07-27 14:27 ` Konke Radlow
2012-07-29 17:07 ` Gregor Jasny [this message]
2012-07-30 9:36 ` Konke Radlow
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=50156DCD.5070205@googlemail.com \
--to=gjasny@googlemail.com \
--cc=kradlow@cisco.com \
--cc=linux-media@vger.kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.