[PATCH v2] groff: Make manpages binary identical

[PATCH v2] groff: Make manpages binary identical

[Jeremy A. Puhlman]

From: Jeremy Puhlman <jpuhlman@mvista.com>

Signed-off-by: Jeremy A. Puhlman <jpuhlman@mvista.com>
---
 .../0001-Make-manpages-mulitlib-identical.patch    | 27 ++++++++++++++++++++++
 meta/recipes-extended/groff/groff_1.22.4.bb        |  1 +
 2 files changed, 28 insertions(+)
 create mode 100644 meta/recipes-extended/groff/files/0001-Make-manpages-mulitlib-identical.patch

diff --git a/meta/recipes-extended/groff/files/0001-Make-manpages-mulitlib-identical.patch b/meta/recipes-extended/groff/files/0001-Make-manpages-mulitlib-identical.patch
new file mode 100644
index 0000000000..329a97cb2e
--- /dev/null
+++ b/meta/recipes-extended/groff/files/0001-Make-manpages-mulitlib-identical.patch
@@ -0,0 +1,27 @@
+From e738f9185ba90f2083c846ade3551234bb5a7cbc Mon Sep 17 00:00:00 2001
+From: Jeremy Puhlman <jpuhlman@mvista.com>
+Date: Sat, 7 Mar 2020 00:59:13 +0000
+Subject: [PATCH] Make manpages mulitlib identical
+
+Upstream-status: Pending
+Signed-off-by: Jeremy Puhlman <jpuhlman@mvista.com>
+---
+ Makefile.am | 2 +-
+ 1 file changed, 1 insertion(+), 1 deletion(-)
+
+diff --git a/Makefile.am b/Makefile.am
+index d18c49b..6175fe9 100644
+--- a/Makefile.am
++++ b/Makefile.am
+@@ -917,7 +917,7 @@ SUFFIXES += .man
+ 	     -e "s|[@]MDATE[@]|`$(PERL) $(top_srcdir)/mdate.pl $<`|g" \
+ 	     -e "s|[@]OLDFONTDIR[@]|`echo $(oldfontdir) | sed -f $(makevarescape)`|g" \
+ 	     -e "s|[@]PDFDOCDIR[@]|`echo $(pdfdocdir) | sed -f $(makevarescape)`|g" \
+-	     -e "s|[@]SYSTEMMACRODIR[@]|`echo $(systemtmacdir) | sed -f $(makevarescape)`|g" \
++	     -e "s|[@]SYSTEMMACRODIR[@]|`echo $(systemtmacdir) | sed -e 's,$(libdir),$(prefix)/lib*,' | sed -f $(makevarescape)`|g" \
+ 	     -e "s|[@]TMAC_AN_PREFIX[@]|$(tmac_an_prefix)|g" \
+ 	     -e "s|[@]TMAC_M_PREFIX[@]|$(tmac_m_prefix)|g" \
+ 	     -e "s|[@]TMAC_MDIR[@]|$(tmacdir)/mm|g" \
+-- 
+2.23.0
+
diff --git a/meta/recipes-extended/groff/groff_1.22.4.bb b/meta/recipes-extended/groff/groff_1.22.4.bb
index 082597f693..e398478349 100644
--- a/meta/recipes-extended/groff/groff_1.22.4.bb
+++ b/meta/recipes-extended/groff/groff_1.22.4.bb
@@ -12,6 +12,7 @@ SRC_URI = "${GNU_MIRROR}/groff/groff-${PV}.tar.gz \
 	file://groff-not-search-fonts-on-build-host.patch \
 	file://0001-support-musl.patch \
 	file://0001-Include-config.h.patch \
+        file://0001-Make-manpages-mulitlib-identical.patch \
 "
 
 SRC_URI[md5sum] = "08fb04335e2f5e73f23ea4c3adbf0c5f"
-- 
2.13.3

✗ patchtest: failure for groff: Make manpages binary identical

[Patchwork]

== Series Details ==

Series: groff: Make manpages binary identical
Revision: 1
URL   : https://patchwork.openembedded.org/series/23145/
State : failure

== Summary ==


Thank you for submitting this patch series to OpenEmbedded Core. This is
an automated response. Several tests have been executed on the proposed
series by patchtest resulting in the following failures:



* Issue             Upstream-Status is in incorrect format [test_upstream_status_presence_format] 
  Suggested fix    Fix Upstream-Status format in 0001-Make-manpages-mulitlib-identical.patch
  Current          Upstream-status: Pending
  Standard format  Upstream-Status: <Valid status>
  Valid status     Pending, Accepted, Backport, Denied, Inappropriate [reason], Submitted [where]



If you believe any of these test results are incorrect, please reply to the
mailing list (openembedded-core@lists.openembedded.org) raising your concerns.
Otherwise we would appreciate you correcting the issues and submitting a new
version of the patchset if applicable. Please ensure you add/increment the
version number when sending the new version (i.e. [PATCH] -> [PATCH v2] ->
[PATCH v3] -> ...).

---
Guidelines:     https://www.openembedded.org/wiki/Commit_Patch_Message_Guidelines
Test framework: http://git.yoctoproject.org/cgit/cgit.cgi/patchtest
Test suite:     http://git.yoctoproject.org/cgit/cgit.cgi/patchtest-oe

Re: ✗ patchtest: failure for groff: Make manpages binary identical

[Richard Purdie]

On Sat, 2020-03-07 at 14:32 +0000, Patchwork wrote:
> == Series Details ==
> 
> Series: groff: Make manpages binary identical
> Revision: 1
> URL   : https://patchwork.openembedded.org/series/23145/
> State : failure
> 
> == Summary ==
> 
> 
> Thank you for submitting this patch series to OpenEmbedded Core. This
> is
> an automated response. Several tests have been executed on the
> proposed
> series by patchtest resulting in the following failures:
> 
> 
> 
> * Issue             Upstream-Status is in incorrect format
> [test_upstream_status_presence_format] 
>   Suggested fix    Fix Upstream-Status format in 0001-Make-manpages-
> mulitlib-identical.patch
>   Current          Upstream-status: Pending
>   Standard format  Upstream-Status: <Valid status>
>   Valid status     Pending, Accepted, Backport, Denied, Inappropriate
> [reason], Submitted [where]
> 

I fixed status -> Status and queued in -next.

Cheers,

Richard

Re: ✗ patchtest: failure for groff: Make manpages binary identical

[Jeremy A. Puhlman]

On 3/7/2020 7:59 AM, Richard Purdie wrote:
> On Sat, 2020-03-07 at 14:32 +0000, Patchwork wrote:
>> == Series Details ==
>>
>> Series: groff: Make manpages binary identical
>> Revision: 1
>> URL   : https://patchwork.openembedded.org/series/23145/
>> State : failure
>>
>> == Summary ==
>>
>>
>> Thank you for submitting this patch series to OpenEmbedded Core. This
>> is
>> an automated response. Several tests have been executed on the
>> proposed
>> series by patchtest resulting in the following failures:
>>
>>
>>
>> * Issue             Upstream-Status is in incorrect format
>> [test_upstream_status_presence_format]
>>    Suggested fix    Fix Upstream-Status format in 0001-Make-manpages-
>> mulitlib-identical.patch
>>    Current          Upstream-status: Pending
>>    Standard format  Upstream-Status: <Valid status>
>>    Valid status     Pending, Accepted, Backport, Denied, Inappropriate
>> [reason], Submitted [where]
>>
> I fixed status -> Status and queued in -next.
Ah sorry about that. I was blind to the case difference. Thank you.
>
> Cheers,
>
> Richard
>
>

-- 
Jeremy A. Puhlman
jpuhlman@mvista.com