Coming changes in trunk
Moderator: Renato
-
- Super User
- Posts: 102
- Joined: Tue May 05, 2009 8:21 am
- Are you a spam bot?: No
- Location: Granbury, TX
Re: Coming changes in trunk
Vadim what do you think about my idea for using wxStandardPaths?
Re: Coming changes in trunk
wxStandardPaths must be used I knew about this class.
Re: Coming changes in trunk
I have already done most of described changes in sources. So, mmex installs in %ProgramFiles%, mmexini.db3 home is %USERPROFILE%. This perfectly works on Windows, but I didn't have enough time to complete this task for Unix. MMEX doesn't work on Unix at the moment. I will fix this issue on this weekends.
Re: Coming changes in trunk
So, I added src\paths.h with helper functions to get locations of mmex's files and dirs. All sources modified to use this new API. Three new implementations of this API added in src\win\platfdep.cpp, src\unix\platfdep.cpp and src\mac\platfdep.cpp. No more #ifdef -s!!!
Mac developers should modify mac\platfdep.cpp in case of some problems with my default implementation. I can't test on Mac. Also you should change build and install scripts for Mac. I did it in bakefile's scripts for win and unix.
Mac developers should modify mac\platfdep.cpp in case of some problems with my default implementation. I can't test on Mac. Also you should change build and install scripts for Mac. I did it in bakefile's scripts for win and unix.
Re: Coming changes in trunk
Also these changes broke mmex's portability (ability to run from flash drive, for example). I will fix portability later. We can just build special "portable" version of mmex or use the same installation and some tricks. This issue is investigated
-
- Super User
- Posts: 112
- Joined: Tue Jul 28, 2009 10:34 pm
- Are you a spam bot?: No
- Location: Atchison, KS
Re: Coming changes in trunk
Wes or I will have to look into this, on mac the ~/Library/Application Support/MoneyManagerEx directory is created with the wrong permissions:
This leads to a can't open database error in mmex.cpp OnInitImpl
Just putting this here as a reminder...
Code: Select all
dr----x--x 2 pom pom 68 Nov 25 08:24 MoneyManagerEx
Code: Select all
inidb.Open(mmex::getPathUser(mmex::SETTINGS));
Re: Coming changes in trunk
I forgot to modify default path for new database. Surely, this should be user's home directory as ~/.mmex on Unix or ~./MoneyManagerEx (?) on Mac.
~/Library/Application Support/MoneyManagerEx must be read-only for user so no error here.
~/Library/Application Support/MoneyManagerEx must be read-only for user so no error here.
-
- Super User
- Posts: 102
- Joined: Tue May 05, 2009 8:21 am
- Are you a spam bot?: No
- Location: Granbury, TX
Re: Coming changes in trunk
we'll probably end up making it ~/Documents or ~/Documents/MMEX. It is best not to use a hidden folder on a mac, that is a filename with a . in front of it, because this is not visible in the default file viewer (Finder). Doing it this way would require the user to open the terminal if they wanted to delete the directory or database.
Re: Coming changes in trunk
In user home directory should be only database and mmexini.db3. All others files should be shared among all users as others applications' files (for Unix, such paths are /usr/lib/mmex, /usr/share/doc/mmex ). In ~/.mmex should resides mmexini.db3 and this normal that this folder is hidden. But database can be created in directory which user will select and default dir can be ~.
-
- Super User
- Posts: 112
- Joined: Tue Jul 28, 2009 10:34 pm
- Are you a spam bot?: No
- Location: Atchison, KS
Re: Coming changes in trunk
I think that ~/Library/Application Support/MoneyManagerEx is fine for the mmexini.db3, but what I'm saying is that the code wouldn't work until I changed the permissions. After adding write permission to the ~/Library/Application Support/MoneyManagerEx folder, it created the inidb and everything was ok.Vadim wrote:I forgot to modify default path for new database. Surely, this should be user's home directory as ~/.mmex on Unix or ~./MoneyManagerEx (?) on Mac.
~/Library/Application Support/MoneyManagerEx must be read-only for user so no error here.