From: Arnout Vandecappelle <arnout@mind.be>
To: buildroot@busybox.net
Subject: [Buildroot] [PATCH 1/2] support/script/scancpan: populates Config.in with homepage
Date: Mon, 16 Jun 2014 20:54:44 +0200 [thread overview]
Message-ID: <539F3D74.4000605@mind.be> (raw)
In-Reply-To: <539F3B3D.9000401@mind.be>
On 06/16/14 20:45, Arnout Vandecappelle wrote:
> On 06/15/14 18:59, Francois Perrad wrote:
>> Signed-off-by: Francois Perrad <francois.perrad@gadz.org>
>
> Reviewed-by: Arnout Vandecappelle (Essensium/Mind) <arnout@mind.be>
Scratch that, I have a comment:
>
> Regards,
> Arnout
>
>> ---
>> support/scripts/scancpan | 5 ++++-
>> 1 file changed, 4 insertions(+), 1 deletion(-)
>>
>> diff --git a/support/scripts/scancpan b/support/scripts/scancpan
>> index d8723f3..bf0aaa5 100755
>> --- a/support/scripts/scancpan
>> +++ b/support/scripts/scancpan
>> @@ -577,6 +577,7 @@ while (my ($distname, $dist) = each %dist) {
>> mkdir $dirname unless -d $dirname;
>> if ($need_target{$distname} && ($force || !-f $cfgname)) {
>> my $abstract = $dist->{abstract};
>> + my $homepage = $dist->{resources}->{homepage} || qq{https://metacpan.org/release/${distname}};
>> say qq{write ${cfgname}} unless $quiet;
>> open my $fh, q{>}, $cfgname;
>> say {$fh} qq{config BR2_PACKAGE_${brname}};
>> @@ -585,8 +586,10 @@ while (my ($distname, $dist) = each %dist) {
>> my $brdep = brname( fsname( $dep ) );
>> say {$fh} qq{\tselect BR2_PACKAGE_${brdep}};
>> }
>> - say {$fh} qq{\thelp} if $abstract;
>> + say {$fh} qq{\thelp};
>> say {$fh} qq{\t ${abstract}} if $abstract;
>> + say {$fh} qq{\t } if $abstract;
There shouldn't be any leading whitespace here, so just make it:
say {$fh} qq{\t ${abstract}\n} if $abstract;
(if that is perlish enough).
>> + say {$fh} qq{\t ${homepage}};
>> close $fh;
>> }
>> if ($force || !-f $mkname) {
>>
>
>
--
Arnout Vandecappelle arnout at mind be
Senior Embedded Software Architect +32-16-286500
Essensium/Mind http://www.mind.be
G.Geenslaan 9, 3001 Leuven, Belgium BE 872 984 063 RPR Leuven
LinkedIn profile: http://www.linkedin.com/in/arnoutvandecappelle
GPG fingerprint: 7CB5 E4CC 6C2E EFD4 6E3D A754 F963 ECAB 2450 2F1F
prev parent reply other threads:[~2014-06-16 18:54 UTC|newest]
Thread overview: 5+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-06-15 16:59 [Buildroot] [PATCH 1/2] support/script/scancpan: populates Config.in with homepage Francois Perrad
2014-06-15 16:59 ` [Buildroot] [PATCH 2/2] perl-mojolicious: new package Francois Perrad
2014-06-16 20:58 ` Arnout Vandecappelle
2014-06-16 18:45 ` [Buildroot] [PATCH 1/2] support/script/scancpan: populates Config.in with homepage Arnout Vandecappelle
2014-06-16 18:54 ` Arnout Vandecappelle [this message]
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=539F3D74.4000605@mind.be \
--to=arnout@mind.be \
--cc=buildroot@busybox.net \
/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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox