Page MenuHomePhabricator

vendor/oyejorge - file permissions trigger debian lintian warnings
Closed, ResolvedPublic

Description

0. Context

DEB packages of mediawiki and many of its extensions, are prepared as part of the WP-MIRROR project.
See https://www.mediawiki.org/wiki/Wp-mirror.

  1. Lintian

During the build process, lintian performs a great number of sanity checks, including seeing if each file has file permissions appropriate to its file type.

When lintian sees a non-executable file with 755 permissions, rather than 644, lintian will throw a warning like:

wp-mirror-mediawiki: executable-not-elf-or-script usr/share/wp-mirror-mediawiki/vendor/oyejorge/less.php/lib/Less/Functions.php

  1. Files which should have permissions set to 644

vendor/oyejorge/less.php/lib/Less/Functions.php
vendor/oyejorge/less.php/lib/Less/Visitor/joinSelector.php
vendor/oyejorge/less.php/lib/Less/Tree/Dimension.php
vendor/oyejorge/less.php/lib/Less/Visitor/import.php
vendor/oyejorge/less.php/lib/Less/Tree/Condition.php
vendor/oyejorge/less.php/lib/Less/Tree/Alpha.php
vendor/oyejorge/less.php/lib/Less/Tree/Operation.php
vendor/oyejorge/less.php/lib/Less/Tree/Selector.php
vendor/oyejorge/less.php/lib/Less/Visitor/processExtends.php
vendor/oyejorge/less.php/lib/Less/Tree/Element.php
vendor/oyejorge/less.php/lib/Less/Visitor/extendFinder.php
vendor/oyejorge/less.php/lib/Less/Tree/Mixin/Definition.php
vendor/oyejorge/less.php/lib/Less/Exception/Parser.php
vendor/oyejorge/less.php/lib/Less/Visitor/toCSS.php
vendor/oyejorge/less.php/lib/Less/Tree/Call.php
vendor/oyejorge/less.php/lib/Less/Tree/Url.php
vendor/oyejorge/less.php/lib/Less/Cache.php
vendor/oyejorge/less.php/lib/Less/Tree/UnicodeDescriptor.php
vendor/oyejorge/less.php/lib/Less/Visitor.php
vendor/oyejorge/less.php/lib/Less/Tree/Anonymous.php
vendor/oyejorge/less.php/lib/Less/Tree/Value.php
vendor/oyejorge/less.php/lib/Less/Tree/Quoted.php
vendor/oyejorge/less.php/lib/Less/Version.php
vendor/oyejorge/less.php/lib/Less/Output/Mapped.php
vendor/oyejorge/less.php/lib/Less/Less.php.combine
vendor/oyejorge/less.php/lib/Less/Tree/Directive.php
vendor/oyejorge/less.php/lib/Less/Tree/Extend.php
vendor/oyejorge/less.php/lib/Less/.easymin/ignore_prefixes
vendor/oyejorge/less.php/lib/Less/Colors.php
vendor/oyejorge/less.php/lib/Less/Tree/Ruleset.php
vendor/oyejorge/less.php/lib/Less/Tree/Comment.php
vendor/oyejorge/less.php/lib/Less/SourceMap/Generator.php
vendor/oyejorge/less.php/lib/Less/Tree/Rule.php
vendor/oyejorge/less.php/lib/Less/Tree/Keyword.php
vendor/oyejorge/less.php/lib/Less/Tree.php
vendor/oyejorge/less.php/lib/Less/Tree/Expression.php
vendor/oyejorge/less.php/lib/Less/Environment.php
vendor/oyejorge/less.php/lib/Less/VisitorReplacing.php
vendor/oyejorge/less.php/lib/Less/Exception/Chunk.php
vendor/oyejorge/less.php/lib/Less/Tree/Color.php
vendor/oyejorge/less.php/lib/Less/Tree/Negative.php
vendor/oyejorge/less.php/lib/Less/Tree/UnitConversions.php
vendor/oyejorge/less.php/lib/Less/Tree/Mixin/Call.php
vendor/oyejorge/less.php/lib/Less/Parser.php
vendor/oyejorge/less.php/lib/Less/Mime.php
vendor/oyejorge/less.php/lib/Less/Tree/Paren.php
vendor/oyejorge/less.php/lib/Less/Configurable.php
vendor/oyejorge/less.php/lib/Less/Tree/NameValue.php
vendor/oyejorge/less.php/lib/Less/Tree/Attribute.php
vendor/oyejorge/less.php/lib/Less/Exception/Compiler.php
vendor/oyejorge/less.php/lib/Less/Tree/Variable.php
vendor/oyejorge/less.php/lib/Less/SourceMap/Base64VLQ.php
vendor/oyejorge/less.php/lib/Less/Tree/Import.php
vendor/oyejorge/less.php/lib/Less/Tree/Javascript.php
vendor/oyejorge/less.php/lib/Less/Tree/Media.php
vendor/oyejorge/less.php/lib/Less/Tree/Assignment.php
vendor/oyejorge/less.php/lib/Less/Tree/Unit.php

  1. Technical details

DEB standards version: 3.9.6
GIT branch: master

Event Timeline

wpmirrordev raised the priority of this task from to Medium.
wpmirrordev updated the task description. (Show Details)
wpmirrordev subscribed.
Legoktm set Security to None.
Legoktm added a subscriber: ori.
Paladox subscribed.

less.php 1.7.0.8 will include the fixes for the files.

Change 240772 had a related patch set uploaded (by Ori.livneh):
oyejorge/less.php 1.7.0.5 → 1.7.0.8

https://gerrit.wikimedia.org/r/240772

Change 240773 had a related patch set uploaded (by Ori.livneh):
oyejorge/less.php 1.7.0.5 → 1.7.0.8

https://gerrit.wikimedia.org/r/240773

Change 240772 merged by jenkins-bot:
oyejorge/less.php 1.7.0.5 → 1.7.0.8

https://gerrit.wikimedia.org/r/240772

Change 240773 merged by jenkins-bot:
oyejorge/less.php 1.7.0.5 → 1.7.0.8

https://gerrit.wikimedia.org/r/240773

I confirm that the above mentioned debian lintian warnings are cured.
Thanks.