[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[bug#27355] [PATCH 3/7] gnu: Add libsigrokdecode.
From: |
Theodoros Foradis |
Subject: |
[bug#27355] [PATCH 3/7] gnu: Add libsigrokdecode. |
Date: |
Mon, 26 Jun 2017 19:50:56 +0300 |
User-agent: |
mu4e 0.9.18; emacs 25.2.1 |
Hello,
> Hi,
>> 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?
>
> Sorry for answering late.
>
> I'm confident that python should be a *normal* input – although it
> should be tested (and the result documented in guix.texi). I'm short in
> time, so I can't test myself now.
>
> From the Documentation [1] I can not tell exactly how libpython behaves
> if used embedded. But the code [2] says:
>
> Step 3 [… search python3 on $PATH]
>
> Step 4. Search the directories pointed to by the preprocessor variables
> PREFIX and EXEC_PREFIX. These are supplied by the Makefile but can be
> passed in as options to the configure script.
>
> So the PREFIX guix is setting is honoured (as a last resort).
>
> BTW: I assume glib should be a normal input, too.
>
> [1] https://docs.python.org/3/c-api/init.html
> [2] https://github.com/python/cpython/blob/master/Modules/getpath.c#L73
>
> [3] https://github.com/python/cpython/blob/master/Modules/getpath.c#L495
> [4] https://github.com/python/cpython/blob/master/Modules/getpath.c#L364
I have tested having either python/glibc as *normal* inputs. Cmake,
which is used by pulseview, gives out an error and the build fails at
the configure phase. Maybe I'm missing something?
Regards,
--
Theodoros Foradis
- [bug#27355] [PATCH 3/7] gnu: Add libsigrokdecode., (continued)
- [bug#27355] [PATCH 3/7] gnu: Add libsigrokdecode., Theodoros Foradis, 2017/06/21
- [bug#27355] [PATCH 3/7] gnu: Add libsigrokdecode., Theodoros Foradis, 2017/06/23
- [bug#27355] [PATCH v2 1/7] gnu: Add libzip., Theodoros Foradis, 2017/06/23
- [bug#27355] [PATCH v2 7/7] gnu: Add pulseview., Theodoros Foradis, 2017/06/23
- [bug#27355] [PATCH v2 2/7] gnu: Add libserialport., Theodoros Foradis, 2017/06/23
- [bug#27355] [PATCH v2 5/7] gnu: Add libsigrok., Theodoros Foradis, 2017/06/23
- [bug#27355] [PATCH v2 3/7] gnu: Add libsigrokdecode., Theodoros Foradis, 2017/06/23
- [bug#27355] [PATCH v2 6/7] gnu: Add sigrok-cli., Theodoros Foradis, 2017/06/23
- [bug#27355] [PATCH v2 4/7] gnu: Add sigrok-firmware-fx2lafw., Theodoros Foradis, 2017/06/23
- [bug#27355] [PATCH 3/7] gnu: Add libsigrokdecode., Hartmut Goebel, 2017/06/26
- [bug#27355] [PATCH 3/7] gnu: Add libsigrokdecode.,
Theodoros Foradis <=
[bug#27355] [PATCH 5/7] gnu: Add libsigrok., Theodoros Foradis, 2017/06/14
[bug#27355] [PATCH 1/7] gnu: Add libzip., Danny Milosavljevic, 2017/06/15
bug#27355: [PATCH 0/7] Add sigrok, Danny Milosavljevic, 2017/06/24