Page MenuHomePhabricator

skins/Tempo - 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:

W: wp-mirror-mediawiki: executable-not-elf-or-script usr/share/wp-mirror-mediawiki/skins/Tempo/i18n/be-tarask.json.

  1. Files which should have permissions set to 644

skins/Tempo/i18n/be-tarask.json
skins/Tempo/i18n/de.json
skins/Tempo/i18n/en.json
skins/Tempo/i18n/frc.json
skins/Tempo/i18n/lb.json
skins/Tempo/i18n/lt.json
skins/Tempo/i18n/mk.json
skins/Tempo/i18n/pl.json
skins/Tempo/i18n/qqq.json
skins/Tempo/i18n/si.json
skins/Tempo/i18n/zh-hans.json
skins/Tempo/i18n/zh-hant.json
skins/Tempo/resources/jquery-1.11.2.min.js
skins/Tempo/resources/tempo.css
skins/Tempo/resources/js/tempo.js
skins/Tempo/templates/NotificationsMenu.tmpl.php
skins/Tempo/README.md
skins/Tempo/Tempo.php
skins/Tempo/Tempo.skin.php

  1. Technical details

DEB standards version: 3.9.6
GIT branch: master

Details

Related Gerrit Patches:
mediawiki/skins/Tempo : masterFix file permissions to pass lintian

Event Timeline

wpmirrordev assigned this task to llbraughler.
wpmirrordev raised the priority of this task from to Normal.
wpmirrordev updated the task description. (Show Details)
wpmirrordev added a project: Other-skins.
wpmirrordev added a subscriber: wpmirrordev.
Restricted Application added a subscriber: Aklapper. · View Herald TranscriptJun 25 2015, 7:21 AM
wpmirrordev set Security to None.

Change 220783 had a related patch set uploaded (by Aklapper):
Fix file permissions to pass lintian

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

Change 220783 merged by Llbraughler:
Fix file permissions to pass lintian

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

llbraughler closed this task as Resolved.Jun 25 2015, 7:21 PM

I have confirmed that the lintian warning, executable-not-elf-or-script, is no longer thrown.
Thanks.