mailarchive of the ptxdist mailing list
 help / color / mirror / Atom feed
From: Michael Olbrich <m.olbrich@pengutronix.de>
To: Robin van der Gracht <robin@protonic.nl>
Cc: ptxdist@pengutronix.de
Subject: Re: [ptxdist] [PATCH 3/3] rules: pcsc-lite: Add option for building with USB support
Date: Wed, 12 Apr 2023 08:32:29 +0200	[thread overview]
Message-ID: <ZDZQfdawS3DxXVLf@pengutronix.de> (raw)
In-Reply-To: <20230406145016.3074015-3-robin@protonic.nl>

On Thu, Apr 06, 2023 at 04:50:16PM +0200, Robin van der Gracht wrote:
> Signed-off-by: Robin van der Gracht <robin@protonic.nl>
> ---
>  rules/pcsc-lite.in   | 5 +++++
>  rules/pcsc-lite.make | 2 +-
>  2 files changed, 6 insertions(+), 1 deletion(-)
> 
> diff --git a/rules/pcsc-lite.in b/rules/pcsc-lite.in
> index 016a70037..de1c288c3 100644
> --- a/rules/pcsc-lite.in
> +++ b/rules/pcsc-lite.in
> @@ -18,6 +18,11 @@ menuconfig PCSC_LITE
>  
>  if PCSC_LITE
>  
> +config PCSC_LITE_USB
> +        bool "USB Support"
> +        help
> +          Add USB support.

Wrong indention. This should be tabs instead of spaces. I've fixed this
while applying the patch. In the future, please run 'ptxdist lint' before
sending patches. It will complain about all the small stuff like this.

Michael

> +
>  config PCSC_LITE_LIBUDEV
>  	bool "libudev support"
>  
> diff --git a/rules/pcsc-lite.make b/rules/pcsc-lite.make
> index d773a8bb6..0c6edcbfe 100644
> --- a/rules/pcsc-lite.make
> +++ b/rules/pcsc-lite.make
> @@ -38,7 +38,7 @@ PCSC_LITE_CONF_OPT := \
>  	$(CROSS_AUTOCONF_USR) \
>  	--$(call ptx/endis, PTXCONF_PCSC_LITE_SYSTEMD_UNIT)-libsystemd \
>  	--disable-serial \
> -	--disable-usb \
> +	--$(call ptx/endis, PTXCONF_PCSC_LITE_USB)-usb \
>  	--$(call ptx/endis, PTXCONF_PCSC_LITE_LIBUDEV)-libudev \
>  	--disable-libusb \
>  	--disable-polkit \
> -- 
> 2.37.2
> 
> 
> 

-- 
Pengutronix e.K.                           |                             |
Steuerwalder Str. 21                       | http://www.pengutronix.de/  |
31137 Hildesheim, Germany                  | Phone: +49-5121-206917-0    |
Amtsgericht Hildesheim, HRA 2686           | Fax:   +49-5121-206917-5555 |



  reply	other threads:[~2023-04-12  6:32 UTC|newest]

Thread overview: 12+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2023-04-06 14:50 [ptxdist] [PATCH 1/3] rules: opensc: Version bumb to 0.23.0 Robin van der Gracht
2023-04-06 14:50 ` [ptxdist] [PATCH 2/3] rules: pcsc-lite: Version bumb to 1.9.8 Robin van der Gracht
2023-04-12  6:29   ` [ptxdist] [APPLIED] " Michael Olbrich
2023-04-06 14:50 ` [ptxdist] [PATCH 3/3] rules: pcsc-lite: Add option for building with USB support Robin van der Gracht
2023-04-12  6:32   ` Michael Olbrich [this message]
2023-04-12  7:56     ` Robin van der Gracht
2023-04-14 10:08       ` Michael Olbrich
2023-04-12  6:28 ` [ptxdist] [PATCH 1/3] rules: opensc: Version bumb to 0.23.0 Michael Olbrich
2023-04-13  7:25   ` Robin van der Gracht
2023-04-13 13:57     ` Michael Olbrich
2023-04-17  8:54       ` Robin van der Gracht
2023-04-14 10:03 ` [ptxdist] [APPLIED] " Michael Olbrich

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=ZDZQfdawS3DxXVLf@pengutronix.de \
    --to=m.olbrich@pengutronix.de \
    --cc=ptxdist@pengutronix.de \
    --cc=robin@protonic.nl \
    /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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox