[oe] [PATCH] socat: Add version 1.7.1.2
John Faith
john at soundmetrics.com
Sat Jan 23 22:31:53 UTC 2010
Paul Menzel wrote:
> Am Samstag, den 23.01.2010, 11:03 -0800 schrieb John Faith:
>
>> ..
>>
>
> Thank you for your patch.
>
> 1. Applying your patch gives me warnings.
>
> $ git am /tmp/\[oe\]_\[PATCH\]_socat\:_Add_version_1.7.1.2
> Applying: socat: Add version 1.7.1.2
> /srv/filme/oe/openembedded/.git/rebase-apply/patch:29: trailing whitespace.
>
> /srv/filme/oe/openembedded/.git/rebase-apply/patch:38: trailing whitespace.
>
> /srv/filme/oe/openembedded/.git/rebase-apply/patch:46: trailing whitespace.
>
> /srv/filme/oe/openembedded/.git/rebase-apply/patch:54: trailing whitespace.
>
> /srv/filme/oe/openembedded/.git/rebase-apply/patch:60: space before tab in indent.
> IF_IP6 ("ipv6only", &opt_ipv6_v6only)
> warning: squelched 13 whitespace errors
> warning: 18 lines add whitespace errors.
>
There are space-tabs in the original source file so I'm not sure how to
remove those. The other trailing whitespace I see is might be an
artifact from diff which I used to create
files/socat-1.7.1.2-cross.patch; I can remove those manually.
> ...
>> diff --git a/recipes/socat/socat_1.7.1.2.bb b/recipes/socat/socat_1.7.1.2.bb
>> new file mode 100644
>> index 0000000..5bbb421
>> --- /dev/null
>> +++ b/recipes/socat/socat_1.7.1.2.bb
>> @@ -0,0 +1,23 @@
>> +SECTION = "console/network"
>> +DEPENDS = "openssl readline"
>> +DESCRIPTION = "Socat is a relay for bidirectional data \
>> +transfer between two independent data channels."
>> +LICENSE = "GPL"
>> +PR = "r1"
>>
>
> You can start with `r0` I think.
>
Ok.
>
>> +
>> +SRC_URI = "http://www.dest-unreach.org/socat/download/socat-1.7.1.2.tar.bz2 \
>>
>
> Can we do this more generic: `…/download/${P}.tar.bz2`.
>
Ok.
>> + file://socat-1.7.1.2-cross.patch;patch=1 "
>> +
>> +inherit autotools
>> +
>> +do_configure() {
>> + # Override this function to avoid autoconf/automake/aclocal/autoheader
>> + # since config.h.in gets recreated.
>> + (cd ${S} && gnu-configize) || die "failure in running gnu-configize"
>> + oe_runconf
>> +}
>> +
>> +do_install_prepend () {
>> + mkdir -p ${D}${bindir}
>> + install -d ${D}${bindir} ${D}${mandir}/man1
>> +}
>>
>
> 2. I cannot build your patch because checksums are missing [1].
>
Ok, I'll add this.
> 3. Looking at the difference between `socat_1.3.2.1.bb` and
> `socat_1.7.1.2.bb` there is not much. Could you factor that out into an
> include file e. g. unify this?
>
I'd rather not potentially break v 1.3 and would prefer not to change this.
> Please fix this up and resend with `[PATCH v2]` in the subject line and
> a note in the commit message.
>
git send-email really doesn't want to put "[PATCH v2]" in the mail
subject even when I specify --subject on the command line. Could you
point me to the trick to do this?
Thanks,
John
More information about the Openembedded-devel
mailing list