]> code.ossystems Code Review - openembedded-core.git/commit
Revert "lib/oe/patch: fix handling of patches with no header"
authorMartin Jansa <martin.jansa@gmail.com>
Sat, 31 Oct 2020 23:45:33 +0000 (00:45 +0100)
committerSteve Sakoman <steve@sakoman.com>
Mon, 2 Nov 2020 14:27:45 +0000 (04:27 -1000)
commit419d45ad6a72c6e296e27f9ae36ddf2a45671bfe
tree904e429083bea6c05fa9c37641c1338d9fcb78c9
parentbbd467c393af5fa34a0b92d5461ff0bf427b3933
Revert "lib/oe/patch: fix handling of patches with no header"

* This reverts commit d9971f5dc8eb7de551fd6f5e058fd24770ef5d78.

* With the missing Subject line fixed in GitApplyTree.prepareCommit()
  we should be able to revert, the fix which was trying to help it by
  parsing GitApplyTree.patch_line_prefix ("%% original patch:") also
  from Subject line, now GitApplyTree.patch_line_prefix should always
  end on separate line which is then skipped when copying the lines to
  resulting patch, see original commit message from Paul:

    lib/oe/patch: fix handling of patches with no header

    If a patch applied by a recipe has no header and we turn the recipe's
    source into a git tree (when PATCHTOOL = "git" or when using devtool
    extract / modify / upgrade), the commit message ends up consisting only
    of the original filename marker ("%% original patch: filename.patch").
    When we come to do turn the commits back into a set of patches in
    extractPatches(), this first line ends up in the "Subject: " part of
    the file, but we were ignoring it because the line didn't start with the
    marker text. The end result was we weren't able to get the original
    patch name. Strip off any "Subject [PATCH x/y]" part before looking for
    the marker text to fix.

    This caused "devtool modify openssl" followed by "devtool update-recipe
    openssl" (without any changes in-between) to remove version-script.patch
    because that patch has no header and we weren't able to determine the
    original filename.

Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
Signed-off-by: Steve Sakoman <steve@sakoman.com>
meta/lib/oe/patch.py