Hi all,

as discussed in the previous mails to avoid syntax errors in the Perl files I 
have
added a new check to Travis. It runs the Perl syntax check (using "perl -w -c") 
for
all *.pm and *.pl files found in the Git repository. The check is active for 
all Git
branches which use the Docker images at Travis. (See [1], [2] if you are 
interested
in the implementation details.)

The check fails when there is a syntax error, if there are just warnings it will
pass. But of course, that does not mean you should not try to fix the warnings 
as well.


I have tested the syntax check in all YaST modules which contain some Perl 
files,
no syntax error was found besides the users module which has just is been fixed 
[3].

But I run the check locally (outside Docker) so the environment might have been 
a bit
different than at Travis. In some cases we might need to install some more Perl
packages there (the syntax check actually loads all used Perl modules) or we 
might
need to adapt the Perl search path (via the PERL5LIB environment variable) to 
find
the source files.

If you need help with that or Travis fails for some other reason then just ping 
me.



[1] https://github.com/yast/docker-yast-ruby/pull/32
[2] https://github.com/yast/docker-yast-cpp/pull/23
[3] https://github.com/yast/yast-users/pull/193


-- 
Ladislav Slezák
YaST Developer

SUSE LINUX, s.r.o.
Corso IIa
Křižíkova 148/34
18600 Praha 8
-- 
To unsubscribe, e-mail: [email protected]
To contact the owner, e-mail: [email protected]

Reply via email to