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

Bug#704197: Please review: systemd checks



On 2013-04-06 18:33, Michael Stapelberg wrote:
> Hi Niels,
> 
> Niels Thykier <niels@thykier.net> writes:
>> [...]
>> Almost; it definitely plugs the issues I mentioned.  That said, I
>> believe we prefer to emit tags instead of erroring out when we see an
>> unexpected file type (e.g. see control-file-is-not-a-file).
>>   Secondly, there is a bug in that link_resolved is only applicable to
>> links.  So if it is not a regular file and not a link, the code will
>> croak in $path->link_resolved[2].
> Okay, so how about this?
> 
> sub check_init_script {
>     my ($pkg, $info, $file) = @_;
> 
>     my $lsb_source_seen;
>     my $path = $info->index ($file);
>     unless ($path->is_regular_file ||
>             ($path->is_symlink && defined($path->link_resolved))) {
>         tag 'init-script-is-not-a-file', $file;

I think you are missing a return here?

>     }
>     open(my $fh, '<', $info->unpacked($file))
>         or fail "cannot open $file: $!";
>     # …
> }
> 
>> It really looks like a implementation of Text::ParseWords's
>> shellwords[3].  If so, we can get that entire sub as a oneliner (we
>> already use Text::ParseWords elsewhere).
> I switched to shellwords. We can always rever to the code we’ve had
> before, but in my tests, shellwords works fine.
> 

Appreciated.

> Find the new files attached.
> 


~Niels


Reply to: