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
prev parent 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