Skip to content

Update Config.php to locate database on Unix#40

Merged
JLiscom merged 1 commit intoJLiscom:masterfrom
pkln:patch-1
Apr 29, 2014
Merged

Update Config.php to locate database on Unix#40
JLiscom merged 1 commit intoJLiscom:masterfrom
pkln:patch-1

Conversation

@pkln
Copy link
Copy Markdown
Contributor

@pkln pkln commented Apr 28, 2014

Added alternative location for sqlite database for use on Unix based systems. Replacing '' with '/'.

Added alternative location for sqlite database for use on Unix based systems. Replacing '\' with '/'.
@JLiscom
Copy link
Copy Markdown
Owner

JLiscom commented Apr 28, 2014

Does the unix path style also work in windows? Do you have ability to test?

@JLiscom
Copy link
Copy Markdown
Owner

JLiscom commented Apr 28, 2014

Also, Do you grant this code(code you are requesting to be pulled) being licensed under a MIT License? The next version will use that license.

@pkln
Copy link
Copy Markdown
Contributor Author

pkln commented Apr 29, 2014

I do not have a Windows system to test now. But from my past experience
on Windows, I suppose it wont work. However, I am not sure if things
change on Windows 8.1 and later.

On 04/28/2014 08:49 AM, FoxUSA wrote:

Does the unix path style also work in windows? Do you have ability to
test?


Reply to this email directly or view it on GitHub
#40 (comment).

-pkln

@pkln
Copy link
Copy Markdown
Contributor Author

pkln commented Apr 29, 2014

I would prefer GPL, but MIT License should be fine.
So, I do grant this be licensed under MIT License.

On 04/28/2014 10:56 PM, FoxUSA wrote:

Also, Do you grant this code(code you are requesting to be pulled)
being licensed under a MIT License? The next version will use that
license.


Reply to this email directly or view it on GitHub
#40 (comment).

-pkln

JLiscom pushed a commit that referenced this pull request Apr 29, 2014
Update Config.php to locate database on Unix
@JLiscom JLiscom merged commit e83d7bd into JLiscom:master Apr 29, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants