From: Theodoros Foradis <theodoros.for@openmailbox.org>
To: 27355@debbugs.gnu.org
Subject: [bug#27355] [PATCH 3/7] gnu: Add libsigrokdecode.
Date: Wed, 21 Jun 2017 23:34:24 +0300 [thread overview]
Message-ID: <87wp85ull4.fsf@openmailbox.org> (raw)
In-Reply-To: <20170619222054.028af180@scratchpost.org>
Hello,
> the README lists some more dependencies:
>
> - doxygen (optional, only needed for the C API docs)
> - graphviz (optional, only needed for the C API docs)
>
> Do you think it makes sense to build the API docs?
>
I recall adding those two as native-inputs but the API docs not
building. I will try again though, and see if I can get it to work. It
makes sense to me to have the docs.
> Also, reading srd.c it seems that this library embeds Python. Is it still supposed to be a propagated-input then? It doesn't need the executable at runtime or anything. Users probably do need the Python standard library, so I guess it's OK... Hartmut, what do you think?
>
libsigrokdecode.pc, that is the pkg-config file lists python in
Requires.private. If python is not propagated, libsigrok which depends
on libsigrokdecode cannot be built.
Maybe there is another way for this to work? I saw that done in another
package.
> Extra Python scripts for libsigrokdecode can be installed by the user into ${XDG_DATA_HOME-.local/share}/libsigrokdecode/decoders which is nice.
>
> Additionally, there's an environment variable SIGROKDECODE_DIR too.
Does the package have to set any native-search-paths for this, to let
the users know of the environment variable?
--
Theodoros Foradis
next prev parent reply other threads:[~2017-06-21 22:47 UTC|newest]
Thread overview: 28+ messages / expand[flat|nested] mbox.gz Atom feed top
2017-06-14 9:54 [bug#27355] [PATCH 0/7] Add sigrok Theodoros Foradis
2017-06-14 9:58 ` [bug#27355] [PATCH 1/7] gnu: Add libzip Theodoros Foradis
2017-06-14 9:58 ` [bug#27355] [PATCH 2/7] gnu: Add libserialport Theodoros Foradis
2017-06-14 9:58 ` [bug#27355] [PATCH 3/7] gnu: Add libsigrokdecode Theodoros Foradis
2017-06-19 20:20 ` Danny Milosavljevic
2017-06-21 20:34 ` Theodoros Foradis [this message]
2017-06-23 21:02 ` Theodoros Foradis
2017-06-23 21:05 ` [bug#27355] [PATCH v2 1/7] gnu: Add libzip Theodoros Foradis
2017-06-23 21:05 ` [bug#27355] [PATCH v2 2/7] gnu: Add libserialport Theodoros Foradis
2017-06-23 21:05 ` [bug#27355] [PATCH v2 3/7] gnu: Add libsigrokdecode Theodoros Foradis
2017-06-23 21:05 ` [bug#27355] [PATCH v2 4/7] gnu: Add sigrok-firmware-fx2lafw Theodoros Foradis
2017-06-23 21:05 ` [bug#27355] [PATCH v2 5/7] gnu: Add libsigrok Theodoros Foradis
2017-06-23 21:05 ` [bug#27355] [PATCH v2 6/7] gnu: Add sigrok-cli Theodoros Foradis
2017-06-23 21:05 ` [bug#27355] [PATCH v2 7/7] gnu: Add pulseview Theodoros Foradis
2017-06-26 10:09 ` [bug#27355] [PATCH 3/7] gnu: Add libsigrokdecode Hartmut Goebel
2017-06-26 16:50 ` Theodoros Foradis
2017-06-14 9:58 ` [bug#27355] [PATCH 4/7] gnu: Add sigrok-firmware-fx2lafw Theodoros Foradis
2017-06-14 9:58 ` [bug#27355] [PATCH 5/7] gnu: Add libsigrok Theodoros Foradis
2017-06-19 20:35 ` Danny Milosavljevic
2017-06-21 23:53 ` Theodoros Foradis
2017-06-14 9:58 ` [bug#27355] [PATCH 6/7] gnu: Add sigrok-cli Theodoros Foradis
2017-06-14 9:58 ` [bug#27355] [PATCH 7/7] gnu: Add pulseview Theodoros Foradis
2017-06-19 20:38 ` Danny Milosavljevic
2017-06-21 23:31 ` Theodoros Foradis
2017-06-24 6:06 ` Danny Milosavljevic
2017-06-15 12:33 ` [bug#27355] [PATCH 1/7] gnu: Add libzip Danny Milosavljevic
2017-06-16 17:42 ` Theodoros Foradis
2017-06-24 6:31 ` bug#27355: [PATCH 0/7] Add sigrok Danny Milosavljevic
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=87wp85ull4.fsf@openmailbox.org \
--to=theodoros.for@openmailbox.org \
--cc=27355@debbugs.gnu.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.
Code repositories for project(s) associated with this external index
https://git.savannah.gnu.org/cgit/guix.git
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.