all messages for Guix-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
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

  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.