patchowork parsing issue?

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hello

I've submitted this patch a couple of times:

  [PATCH] media_build: support building against a specific kernel

I formatted it with git-format-patch and it makes it into patchwork:

  https://patchwork.linuxtv.org/patch/60186/
  https://patchwork.linuxtv.org/patch/59979/

but it doesn't look right when it gets there. For some reason,
the last hunk of the patch is included with the patch description.
This was why I resent it the first time, I thought I must have
messed up the formatting.

Admittedly I'm sending from gmail's web interface but I have taken
some care to get the text correctly formatted and patchwork did the
same thing with it twice, so I'm starting to wonder if there's an issue
with patchwork.

I've attached my copy of the last outgoing email attempt in case
that's useful. The only thing that looks suspicous to me is on the
signed-off-by line I did not enclose my email address in '< >' markers,
while that last hunk happens to have those characters in it.

Kind regards
Vince
From vincent.mcintyre@xxxxxxxxx Fri Nov 15 22:11:14 2019
MIME-Version: 1.0
Received: by 2002:a19:905:0:0:0:0:0 with HTTP; Fri, 15 Nov 2019 03:11:14 -0800 (PST)
Date: Fri, 15 Nov 2019 22:11:14 +1100
Delivered-To: vincent.mcintyre@xxxxxxxxx
Message-ID: <CAEsFdVP9FkvG8+oPLkMUuaxPNYnc2Ug1dBgnAAxZqS_fUGf77Q@xxxxxxxxxxxxxx>
Subject: [PATCH] media_build: support building against a specific kernel.
From: Vincent McIntyre <vincent.mcintyre@xxxxxxxxx>
To: linux-media <linux-media@xxxxxxxxxxxxxxx>
Content-Type: text/plain; charset="UTF-8"
Status: RO
Content-Length: 2188
Lines: 73

I find it helpful to be able to build the linux-media modules
for a kernel other than the one I am currently running.

This is a resend of the patch that appeared at
  https://patchwork.linuxtv.org/patch/59979/
and completely supercedes that patch.
There are no differences in the patch, just the formatting of the
submission email.

Signed-off-by: vincent.mcintyre@xxxxxxxxx

---
 build | 16 ++++++++++++++--
 1 file changed, 14 insertions(+), 2 deletions(-)

diff --git a/build b/build
index 2f6b3f4..c2067e7 100755
--- a/build
+++ b/build
@@ -14,6 +14,7 @@ my $check_only = 0;
 my $main_git = 0;
 my $depth;
 my @git;
+my $kernel_version = '';

 my $main_git_url = "git://linuxtv.org/media_tree.git";
 my $main_branch = "master";
@@ -27,6 +28,7 @@ GetOptions('v|verbose' => \$level,
           'main_git|main-git' => \$main_git,
           'depth=i' => \$depth,
           'git=s{2}' => \@git,
+          'version=s' => \$kernel_version,
          ) or pod2usage(2);
 pod2usage(1) if $help;
 pod2usage(-exitstatus => 0, -verbose => 2) if $man;
@@ -522,7 +524,12 @@ print "******************\n";
 print "* Start building *\n";
 print "******************\n";

-run("make allyesconfig", "can't select all drivers");
+if ($kernel_version =~ /\S/) {
+    print "Building for kernel version $kernel_version\n";
+    run("make allyesconfig VER=$kernel_version", "can't select all drivers");
+} else {
+    run("make allyesconfig", "can't select all drivers");
+}
 run("make", "build failed");

 print "**********************************************************\n";
@@ -539,7 +546,7 @@ build - Builds the media drivers without needing
to compile a new kernel
 =head1 SYNOPSIS

 build [--help] [--man] [--verbose] [--check-only] [<--git> [URL] [BRANCH]]
- [--main-git] [--depth [DEPTH]]
+ [--main-git] [--depth [DEPTH]] [--version [KERNELVERSION]]

 =head1 OPTIONS

@@ -579,6 +586,11 @@ depth parameter, in order to get a smaller tree.

 That helps to reduce disk storage and download time.

+=item B<--version> [KERNELVERSION]
+
+Try to build for a particular kernel version. The format of the
+version string is the same as the output from 'uname -r'.
+
 =back

 =head1 DESCRIPTION
-- 
2.7.4


[Index of Archives]     [Linux Input]     [Video for Linux]     [Gstreamer Embedded]     [Mplayer Users]     [Linux USB Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Yosemite Backpacking]

  Powered by Linux