All of lore.kernel.org
 help / color / mirror / Atom feed
From: Richard Purdie <richard.purdie@linuxfoundation.org>
To: Patches and discussions about the oe-core layer
	<openembedded-core@lists.openembedded.org>
Cc: Scott Garman <scott.a.garman@intel.com>
Subject: Re: [PATCH] docbook-utils-native: fix syntax problem in jw.in
Date: Thu, 29 Mar 2012 22:28:09 +0100	[thread overview]
Message-ID: <1333056489.18082.31.camel@ted> (raw)
In-Reply-To: <1333004316-6877-1-git-send-email-sledz@dresearch-fe.de>

On Thu, 2012-03-29 at 08:58 +0200, Steffen Sledz wrote:
> Fix runtime error occurred e.g. with docbook-to-man calls:
> 
>   grep: character class syntax is [[:space:]], not [:space:]
>   grep: character class syntax is [[:space:]], not [:space:]
>   jw: There is no frontend called "/docbook/utils-0.6.14/frontends/docbook".
> 
> See also:
> 
>    <https://qa.mandriva.com/show_bug.cgi?id=61127>
> 
> Signed-off-by: Steffen Sledz <sledz@dresearch-fe.de>
> ---
>  .../docbook-utils/docbook-utils-0.6.14/re.patch    |   15 +++++++++++++++
>  .../docbook-utils/docbook-utils-native_0.6.14.bb   |    7 +++++--
>  2 files changed, 20 insertions(+), 2 deletions(-)
>  create mode 100644 meta/recipes-devtools/docbook-utils/docbook-utils-0.6.14/re.patch
> 
> diff --git a/meta/recipes-devtools/docbook-utils/docbook-utils-0.6.14/re.patch b/meta/recipes-devtools/docbook-utils/docbook-utils-0.6.14/re.patch
> new file mode 100644
> index 0000000..1e53389
> --- /dev/null
> +++ b/meta/recipes-devtools/docbook-utils/docbook-utils-0.6.14/re.patch
> @@ -0,0 +1,15 @@


There is no header on this patch saying what the patch does, why, where
it came from or what its status is. Please resubmit with a header added:

https://wiki.yoctoproject.org/wiki/Contribution_Guidelines#Patch_Header_Recommendations

This is so when we look at the patch a year from now, we can understand
why its there.

Cheers,

Richard





  reply	other threads:[~2012-03-29 21:37 UTC|newest]

Thread overview: 5+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-03-29  6:58 [PATCH] docbook-utils-native: fix syntax problem in jw.in Steffen Sledz
2012-03-29 21:28 ` Richard Purdie [this message]
2012-03-30  9:45   ` [PATCH v2] " Steffen Sledz
2012-04-03  9:56     ` Steffen Sledz
2012-04-03 14:19       ` Richard Purdie

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=1333056489.18082.31.camel@ted \
    --to=richard.purdie@linuxfoundation.org \
    --cc=openembedded-core@lists.openembedded.org \
    --cc=scott.a.garman@intel.com \
    /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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.