Bug 175214 - directories not created in /var before attempting to chown them
Summary: directories not created in /var before attempting to chown them
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: Fedora
Classification: Fedora
Component: tinyerp
Version: 4
Hardware: i386
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Dan Horák
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2005-12-07 20:00 UTC by Stephen Biggs
Modified: 2007-11-30 22:11 UTC (History)
1 user (show)

Fixed In Version: 3.1.1-5
Clone Of:
Environment:
Last Closed: 2005-12-11 12:29:39 UTC
Type: ---
Embargoed:


Attachments (Terms of Use)

Description Stephen Biggs 2005-12-07 20:00:25 UTC
From Bugzilla Helper:
User-Agent: Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.7.12) Gecko/20050926 Fedora/1.0.7-1.1.fc4.swupdate Firefox/1.0.7

Description of problem:
Directories not created correctly at install of this package in /var.

Version-Release number of selected component (if applicable):
tinyerp-server-3.1.1-1.fc4

How reproducible:
Always

Steps to Reproduce:
1. Install tinyerp-server-3.1.1-1.fc4
2.
3.
  

Actual Results:  When installing this on a clean system, in the preinstall script there is the 
line:
chown -R tinyerp:tinyerp /var/log/tinyerp /var/spool/tinyerp

This fails with a complaining error since these directories don't exist.

Expected Results:  Installation without error messages.

Additional info:

Before the chown command, the line:
mkdir -p  /var/log/tinyerp /var/spool/tinyerp

needs to be inserted inthe preinstall script.

Once these directories already exist, installation of the package completes without error or other incident.

Comment 1 Dan Horák 2005-12-08 16:08:05 UTC
The dirs are included in the tinyerp-server package, but the chown command
should be executed in %post script instead of %pre. In the time of executing the
%pre script they don't exist yet. Fixed in tinyerp-server-3.1.1-2.

Comment 2 Dan Horák 2005-12-08 17:28:33 UTC
The chown command is not even required, ownership is set via %attr in the spec
file. I have released tinyerp-3.1.1-3.

Comment 3 Dan Horák 2005-12-11 12:29:39 UTC
I have fixed dependencies for %pre and %postun scripts. Will be in tinyerp-3.1.1-5.


Note You need to log in before you can comment on or make changes to this bug.