Bug 1039943

Summary: ship /var/lib/glpi/files/_pictures
Product: [Fedora] Fedora Reporter: albert <superber>
Component: glpiAssignee: Remi Collet <fedora>
Status: CLOSED DEFERRED QA Contact: Fedora Extras Quality Assurance <extras-qa>
Severity: unspecified Docs Contact:
Priority: unspecified    
Version: rawhideCC: fedora
Target Milestone: ---   
Target Release: ---   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2013-12-11 09:24:20 UTC Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:

Description albert 2013-12-10 11:44:22 UTC
Description of problem:

/var/lib/glpi/files/_pictures is defined but not bundled

Version-Release number of selected component (if applicable):

glpi-0.84.3-1.fc21.noarch.rpm

How reproducible:

#rpm -ql glpi  | grep /var/lib/glpi/files/_
/var/lib/glpi/files/_cache
/var/lib/glpi/files/_cron
/var/lib/glpi/files/_dumps
/var/lib/glpi/files/_graphs
/var/lib/glpi/files/_lock
/var/lib/glpi/files/_plugins
/var/lib/glpi/files/_rss
/var/lib/glpi/files/_sessions
/var/lib/glpi/files/_tmp
/var/lib/glpi/files/_uploads

#fgrep /var/lib/glpi/files/_ /usr/share/glpi/config/config_path.php  
define('GLPI_DUMP_DIR',       '/var/lib/glpi/files/_dumps');
define('GLPI_CACHE_DIR',      '/var/lib/glpi/files/_cache/');
define('GLPI_CRON_DIR',       '/var/lib/glpi/files/_cron');
define('GLPI_SESSION_DIR',    '/var/lib/glpi/files/_sessions');
define('GLPI_PLUGIN_DOC_DIR', '/var/lib/glpi/files/_plugins');
define('GLPI_LOCK_DIR',       '/var/lib/glpi/files/_lock/');
define('GLPI_GRAPH_DIR',      '/var/lib/glpi/files/_graphs/');
define('GLPI_PICTURE_DIR',    '/var/lib/glpi/files/_pictures/');   <--- missing
define('GLPI_TMP_DIR',        '/var/lib/glpi/files/_tmp/');
define('GLPI_RSS_DIR',        '/var/lib/glpi/files/_rss/');
define('GLPI_UPLOAD_DIR',     '/var/lib/glpi/files/_uploads/');

Comment 1 Remi Collet 2013-12-11 09:24:20 UTC
This is a small error, but without any impact.

The _pictures dir is only used in glpi 0.85.
Having this define is not needed but will simply not be used.

I will remove the define in a future update.