mailarchive of the ptxdist mailing list
 help / color / mirror / Atom feed
From: Roland Hieber <rhi@pengutronix.de>
To: Juergen Borleis <juergen@kreuzholzen.de>
Cc: ptxdist@pengutronix.de
Subject: Re: [ptxdist] [PATCH] libpaper: Make the default paper configurable
Date: Mon, 7 Jan 2019 11:53:27 +0100	[thread overview]
Message-ID: <20190107105327.bfampbm5vo6pwm2i@pengutronix.de> (raw)
In-Reply-To: <20190104115239.9102-1-jbe@pengutronix.de>

Hi Jürgen,

I would suggest --with-default-paper instead of --with-paper, see the
following fixup. Otherwise looks fine to me.

 - Roland

diff --git a/patches/libpaper-1.1.24+nmu5/0001-Make-the-default-paper-configurable.patch b/patches/libpaper-1.1.24+nmu5/0001-Make-the-default-paper-configurable.patch
index 8a71f9009..bcc61dd23 100644
--- a/patches/libpaper-1.1.24+nmu5/0001-Make-the-default-paper-configurable.patch
+++ b/patches/libpaper-1.1.24+nmu5/0001-Make-the-default-paper-configurable.patch
@@ -11,7 +11,7 @@ Signed-off-by: Juergen Borleis <juergen@kreuzholzen.de>
  1 file changed, 8 insertions(+), 1 deletion(-)
 
 diff --git a/configure.ac b/configure.ac
-index b6e937163775..b8744604629b 100644
+index 5451156fafb5..4f9c53df37eb 100644
 --- a/configure.ac
 +++ b/configure.ac
 @@ -30,7 +30,14 @@ AC_SUBST(PAPERCONFVAR)
@@ -20,13 +20,13 @@ index b6e937163775..b8744604629b 100644
  
 -PAPERSIZE=letter
 +AC_MSG_CHECKING([which paper format name is the default one])
-+AC_ARG_WITH([paper],
-+	[AS_HELP_STRING([--with-paper=@<:@name@:>@], [build with default paper name @<:@default=letter@:>@])],
++AC_ARG_WITH([default-paper],
++	[AS_HELP_STRING([--with-default-paper=@<:@name@:>@], [build with default paper name @<:@default=letter@:>@])],
 +	[],
-+	[with_paper=letter])
-+AC_MSG_RESULT([${with_paper}])
++	[with_default_paper=letter])
++AC_MSG_RESULT([${with_default_paper}])
 +
-+PAPERSIZE=${with_paper}
++PAPERSIZE=${with_default_paper}
  AC_SUBST(PAPERSIZE)
  AH_TEMPLATE(PAPERSIZE, [Default paper size])
  AC_DEFINE_UNQUOTED(PAPERSIZE, "$PAPERSIZE")
diff --git a/rules/libpaper.make b/rules/libpaper.make
index 453a3aae1..45cd946cc 100644
--- a/rules/libpaper.make
+++ b/rules/libpaper.make
@@ -39,7 +39,7 @@ LIBPAPER_LICENSE_FILES	:= file://COPYING;md5=0278281246c1e59af1ef0ae1784a4948
 LIBPAPER_CONF_TOOL	:= autoconf
 LIBPAPER_CONF_OPT	:= \
 	$(CROSS_AUTOCONF_USR) \
-	--with-paper=$(PTXCONF_LIBPAPER_SIZE)
+	--with-default-paper=$(PTXCONF_LIBPAPER_SIZE)
 
 #
 # ----------------------------------------------------------------------------


_______________________________________________
ptxdist mailing list
ptxdist@pengutronix.de

      reply	other threads:[~2019-01-08 13:51 UTC|newest]

Thread overview: 2+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2019-01-04 11:52 Juergen Borleis
2019-01-07 10:53 ` Roland Hieber [this message]

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=20190107105327.bfampbm5vo6pwm2i@pengutronix.de \
    --to=rhi@pengutronix.de \
    --cc=juergen@kreuzholzen.de \
    --cc=ptxdist@pengutronix.de \
    /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