[yast-devel] Added Perl syntax check at Travis
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: yast-devel+unsubscribe@opensuse.org To contact the owner, e-mail: yast-devel+owner@opensuse.org
participants (1)
-
Ladislav Slezak