Re: [PATCH 1/2] MODSIGN: perlify sign-file and merge in x509keyid [ver #2]

2012-10-19 Thread Linus Torvalds
On Fri, Oct 19, 2012 at 3:58 PM, Linus Torvalds wrote: > > Also, I wonder how standard this /usr/bin/perl is: > >> +#!/usr/bin/perl -w Well, we have other uses of that path in the kernel already, and your v3 patch uses 'perl' in the Makefile to run it, so it will pick up the path that way. So I

Re: [PATCH 1/2] MODSIGN: perlify sign-file and merge in x509keyid [ver #2]

2012-10-19 Thread Linus Torvalds
This version still seems to have some debug turds in it: On Fri, Oct 19, 2012 at 3:51 PM, David Howells wrote: > > + printk("Check for sig\n"); > + ... > + printk("Found sig\n"); ... > +#define DEBUG Also, I wonder how standard this /usr/bin/perl is: >

[PATCH 1/2] MODSIGN: perlify sign-file and merge in x509keyid [ver #3]

2012-10-19 Thread David Howells
Turn sign-file into perl and merge in x509keyid. The latter doesn't need to be a separate script as it doesn't actually need to work out the SHA1 sum of the X.509 certificate itself, since it can get that from the X.509 certificate. Signed-off-by: David Howells --- Makefile |2

Re: [PATCH 1/2] MODSIGN: perlify sign-file and merge in x509keyid [ver #2]

2012-10-19 Thread David Howells
David Howells wrote: > + printk("Check for sig\n"); > + > ... > + printk("Found sig\n"); > ... > +#define DEBUG And I also forgot to remove the debugging. Sigh. David -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to

[PATCH 1/2] MODSIGN: perlify sign-file and merge in x509keyid [ver #2]

2012-10-19 Thread David Howells
Turn sign-file into perl and merge in x509keyid. The latter doesn't need to be a separate script as it doesn't actually need to work out the SHA1 sum of the X.509 certificate itself, since it can get that from the X.509 certificate. Signed-off-by: David Howells --- Makefile|

Re: [PATCH 1/2] MODSIGN: perlify sign-file and merge in x509keyid

2012-10-19 Thread David Howells
David Howells wrote: > -mod_sign_cmd = sh $(srctree)/scripts/sign-file $(MODSECKEY) $(MODPUBKEY) > $(srctree)/scripts/x509keyid > +mod_sign_cmd = sh $(srctree)/scripts/sign-file $(MODSECKEY) $(MODPUBKEY) Hmmm... That's not quite right. That needs to be 'perl' not 'sh'. David -- To

[PATCH 1/2] MODSIGN: perlify sign-file and merge in x509keyid

2012-10-19 Thread David Howells
Turn sign-file into perl and merge in x509keyid. The latter doesn't need to be a separate script as it doesn't actually need to work out the SHA1 sum of the X.509 certificate itself, since it can get that from the X.509 certificate. Signed-off-by: David Howells --- Makefile|

[PATCH 1/2] MODSIGN: perlify sign-file and merge in x509keyid

2012-10-19 Thread David Howells
Turn sign-file into perl and merge in x509keyid. The latter doesn't need to be a separate script as it doesn't actually need to work out the SHA1 sum of the X.509 certificate itself, since it can get that from the X.509 certificate. Signed-off-by: David Howells dhowe...@redhat.com --- Makefile

Re: [PATCH 1/2] MODSIGN: perlify sign-file and merge in x509keyid

2012-10-19 Thread David Howells
David Howells dhowe...@redhat.com wrote: -mod_sign_cmd = sh $(srctree)/scripts/sign-file $(MODSECKEY) $(MODPUBKEY) $(srctree)/scripts/x509keyid +mod_sign_cmd = sh $(srctree)/scripts/sign-file $(MODSECKEY) $(MODPUBKEY) Hmmm... That's not quite right. That needs to be 'perl' not 'sh'. David

[PATCH 1/2] MODSIGN: perlify sign-file and merge in x509keyid [ver #2]

2012-10-19 Thread David Howells
Turn sign-file into perl and merge in x509keyid. The latter doesn't need to be a separate script as it doesn't actually need to work out the SHA1 sum of the X.509 certificate itself, since it can get that from the X.509 certificate. Signed-off-by: David Howells dhowe...@redhat.com --- Makefile

Re: [PATCH 1/2] MODSIGN: perlify sign-file and merge in x509keyid [ver #2]

2012-10-19 Thread David Howells
David Howells dhowe...@redhat.com wrote: + printk(Check for sig\n); + ... + printk(Found sig\n); ... +#define DEBUG And I also forgot to remove the debugging. Sigh. David -- To unsubscribe from this list: send the line unsubscribe linux-kernel in the body of a

[PATCH 1/2] MODSIGN: perlify sign-file and merge in x509keyid [ver #3]

2012-10-19 Thread David Howells
Turn sign-file into perl and merge in x509keyid. The latter doesn't need to be a separate script as it doesn't actually need to work out the SHA1 sum of the X.509 certificate itself, since it can get that from the X.509 certificate. Signed-off-by: David Howells dhowe...@redhat.com --- Makefile

Re: [PATCH 1/2] MODSIGN: perlify sign-file and merge in x509keyid [ver #2]

2012-10-19 Thread Linus Torvalds
This version still seems to have some debug turds in it: On Fri, Oct 19, 2012 at 3:51 PM, David Howells dhowe...@redhat.com wrote: + printk(Check for sig\n); + ... + printk(Found sig\n); ... +#define DEBUG Also, I wonder how standard this /usr/bin/perl is:

Re: [PATCH 1/2] MODSIGN: perlify sign-file and merge in x509keyid [ver #2]

2012-10-19 Thread Linus Torvalds
On Fri, Oct 19, 2012 at 3:58 PM, Linus Torvalds torva...@linux-foundation.org wrote: Also, I wonder how standard this /usr/bin/perl is: +#!/usr/bin/perl -w Well, we have other uses of that path in the kernel already, and your v3 patch uses 'perl' in the Makefile to run it, so it will pick up