gitprep
gitprep copied to clipboard
Suggested better setup
Hi,
I think these steps will make it easier to install and upgrade:
- rename
gitprep.confasgitprep.conf.example - remove
/datafolder - Installation should start with
git clone https://github.com/yuki-kimoto/gitprepinstead of usingcurlgit clone https://github.com/yuki-kimoto/gitprep cd gitprep ./setup_module - If
gitprep.confdoes not exists,setup_modulewill create it fromgitprep.conf.example - If
/datafolder does not exists,setup_moduleorsetup_databasewill create it - To upgrade, just do a
git pullin the gitprep folder (which won't overwritegitprep.confand/data)
Possible?
OK.
If /data folder does not exists, setup_module or setup_database will create it
Are you going to choose setup_module or setup_database?
I haven't look at the code to determine where is the better place. Actually, why is there a need to breakdown into setup_module and setup_database?
The reason is that these are different operations.
It is OK that creating the setup script and run setup_module and setup_database in the setup script.
It is OK that creating the C script and run C<setup_module> and C<setup_database> in the C script.
Yes, I also thought of this, but you are faster haha.
let's call it setup
- Create
gitprep.confif not exists - Create
/dataif not exists - run
setup_module - run
setup_database
How about that?
Almost OK.
How about doing the following?
- run setup_module
- Create gitprep.conf if not exists
- run setup_database(Create /data if not exists)
Yes, I'm OK, you are the boss :)
I can merge this changes if the codes are written.
OK
/http @Disis PO COMMET blick Tets?
Please consider assigning executable permissions to all setup scripts, including setup, setup_config, and setup_script. Further, please note that the other ones, setup_database and setup_module, are not executable or readable for all users. Please consider setting permissions mode to 0775.
Due especially to calls among various scripts, operation is broken without appropriate permissions.
chmod 755 setup setup_config setup_database setup_module setup_script
chmod 755 setup setup_config setup_database setup_module setup_script
Done in 75813b2. This issue can be closed now.