Anonymous | Login | 2024-08-24 21:01 EDT | ![]() |
Main | My View | View Issues | Change Log | Roadmap |
View Issue Details [ Jump to Notes ] | [ Issue History ] [ Print ] | |||||||
ID | Project | Category | View Status | Date Submitted | Last Update | |||
0001038 | PacketFence | packaging | public | 2010-07-27 10:44 | 2012-08-07 10:10 | |||
Reporter | obilodeau | |||||||
Assigned To | obilodeau | |||||||
Priority | normal | Severity | minor | Reproducibility | sometimes | |||
Status | closed | Resolution | fixed | |||||
Platform | OS | OS Version | ||||||
Product Version | ||||||||
Target Version | Fixed in Version | 3.5.0 | ||||||
Summary | 0001038: Permission changing in installer.pl | |||||||
Description | Why isn't it done through the RPM? Here's a typical problematic scenario: MySQL db was installed somewhere in /usr/local/pf/. Running installer.pl chown all the files to pf:pf which makes mysqld sad. Some mailing list conversation backing the change: > Now i have seen, that the new installer.pl script > makes at the end (after the mysql database creation) a chown to the > installation directory. > So it has changed my permission to the mysql database directory. It's nothing new, as far as I could go in our source repo, it always worked like that. > > Was it a bad idea to install the mysql database in the pf directory? Yes and no, we know we like to keep stuff in /var/lib/mysql/ since it's the standard and others don't look around for the database in emergency situations. But it should not prevent you of doing what you think is fine. > > Or is it possible to change the installer script, that only the > installed files/dir are changed? > - Another simple solution is to print a notification in the installation > process. > - Or make a notice in the Documentation section for the HA when it was a > good idea to include the mysql database in the pf directory. > Problem is I don't even know why we set everything to be owned by pf:pf in that script and we don't let the RPM take care of it instead.. Also, on upgrade, you don't need to run installer.pl. That's another thing that is unclear. Some code to be rethought of: # TODO: au démarrage de PF, créer les logs si les fichiers n'existent pas # et supprimer cette section print "Creating empty log files\n"; `touch $install_dir/logs/packetfence.log`; `touch $install_dir/logs/snmptrapd.log`; `touch $install_dir/logs/access_log`; `touch $install_dir/logs/error_log`; `touch $install_dir/logs/admin_access_log`; `touch $install_dir/logs/admin_error_log`; `touch $install_dir/logs/admin_debug_log`; `touch $install_dir/logs/pfdetect`; `touch $install_dir/logs/pfmon`; `touch $install_dir/logs/pfredirect`; print "Setting permissions\n"; print " Chowning $install_dir pf:pf\n"; `chown -R pf:pf $install_dir`; foreach my $file (@suids) { print " Chowning $file root:root and setting SGID bits\n"; `chown root:root $file`; `chmod 6755 $file`; } | |||||||
Tags | No tags attached. | |||||||
fixed in git revision | ||||||||
fixed in mtn revision | ||||||||
Attached Files | ||||||||
![]() |
|||||||||||
|
![]() |
|
(0002862) dwuelfrath (administrator) 2012-08-03 09:56 |
Since PacketFence 3.5.0, installer.pl and configurator.pl scripts have been removed. A new web based configurator has been introduced. Feel free to reopen in any case. |
(0002896) obilodeau (reporter) 2012-08-07 10:10 |
I'm considering the removal as a fix and not as an indication that it won't be fixed. I changed the resolution accordingly. ( IOW: don't mess with my developer stats! :P ) |
![]() |
|||
Date Modified | Username | Field | Change |
2010-07-27 10:44 | obilodeau | New Issue | |
2010-07-27 10:44 | obilodeau | Status | new => assigned |
2010-07-27 10:44 | obilodeau | Assigned To | => obilodeau |
2010-09-15 11:29 | obilodeau | Target Version | 1.9.1 => 1.9.2 |
2010-09-22 16:02 | obilodeau | Target Version | 1.9.2 => 1.9.3 |
2012-05-07 15:38 | obilodeau | Relationship added | child of 0001444 |
2012-08-03 09:56 | dwuelfrath | Note Added: 0002862 | |
2012-08-03 09:56 | dwuelfrath | Status | assigned => closed |
2012-08-03 09:56 | dwuelfrath | Resolution | open => won't fix |
2012-08-03 09:56 | dwuelfrath | Fixed in Version | => 3.5.0 |
2012-08-03 09:56 | dwuelfrath | Target Version | 1.9.3 => |
2012-08-07 10:10 | obilodeau | Note Added: 0002896 | |
2012-08-07 10:10 | obilodeau | Resolution | won't fix => fixed |
2012-08-07 10:31 | obilodeau | Relationship added | related to 0001503 |
Copyright © 2000 - 2012 MantisBT Group |