FOSSology

The missing postgresql-client apparently is not an issue. However, this script should be checked for accuracy anyway. In particular, please verify it checks and corrects permissions on the repo directories.

The following code (from fo-postinstall) shows the repo directory gets created (if it doesn't exist) with fossy:fossy and 2770. If the directory already exists, nothing is done. Do you think it's necessary to insure ownership & permissions are correct for existing repos?

if [ ! -d "$REPO" ] ; then
     mkdir -p "$REPO" 
     chown {$PROJECTUSER}:{$PROJECTGROUP} $REPO
     chmod 2770 $REPO
   # make sure the parent dir has the right permissions too
     chown {$PROJECTUSER}:{$PROJECTGROUP} $\{REPO%/*\}
     chmod 2770 $\{REPO%/*\}
  else
     echo "NOTE: Repository already exists at $REPO" 
  fi

I chatted with Bob about this. He suggests (and I agree) that fo-postinstall should check an existing repo directories ownership & permissions and issue a warning message if they are wrong. ALSO, the warning should instruct the user to re-run fo-postinstall with a "force" option to change an existing repo's ownership/permissions, if they are incorrect.

reassigning to Alex.

Updating to a high defect and setting iteration 7 to be addressed. Reassigning to Mary to validate if this has already been fixed.

set estimate to 1

The original title & description for this issue was wrong (see note #2). The real issue is described in note #5:

"I chatted with Bob about this. He suggests (and I agree) that fo-postinstall should check an existing repo directories ownership & permissions and issue a warning message if they are wrong. ALSO, the warning should instruct the user to re-run fo-postinstall with a "force" option to change an existing repo's ownership/permissions, if they are incorrect."

I am changing the priority to normal and removing from iteration 7. This issue should be re-evaluated for inclusion in 2.1.0