[GitHub] cordova-plugin-file pull request: updated support for the OS X pla...

2016-04-07 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/cordova-plugin-file/pull/42 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the fea

[GitHub] cordova-plugin-file pull request: updated support for the OS X pla...

2016-04-07 Thread tripodsan
Github user tripodsan commented on the pull request: https://github.com/apache/cordova-plugin-file/pull/42#issuecomment-207138136 please close. we got osx support in the meantime added. --- If your project is set up for it, you can reply to this email and have your reply appear on Git

[GitHub] cordova-plugin-file pull request: updated support for the OS X pla...

2016-04-07 Thread purplecabbage
Github user purplecabbage commented on the pull request: https://github.com/apache/cordova-plugin-file/pull/42#issuecomment-207121298 This pr is hopelessly out of date. If it is still relevant, please rebase it, otherwise I will be closing it shortly. --- If your project is set up fo

[GitHub] cordova-plugin-file pull request: updated support for the OS X pla...

2014-08-13 Thread jsoref
Github user jsoref commented on the pull request: https://github.com/apache/cordova-plugin-file/pull/42#issuecomment-52084825 @onflapp please see: http://wiki.apache.org/cordova/ContributorWorkflow (ICLA) http://wiki.apache.org/cordova/IssueWorkflow (file a bug in jira; include

[GitHub] cordova-plugin-file pull request: updated support for the OS X pla...

2014-05-02 Thread shazron
Github user shazron commented on a diff in the pull request: https://github.com/apache/cordova-plugin-file/pull/42#discussion_r12221118 --- Diff: src/ios/CDVFile.m --- @@ -313,11 +313,9 @@ - (void)pluginInitialize --- End diff -- Since you are doing the copy route

[GitHub] cordova-plugin-file pull request: updated support for the OS X pla...

2014-05-02 Thread shazron
Github user shazron commented on a diff in the pull request: https://github.com/apache/cordova-plugin-file/pull/42#discussion_r12221112 --- Diff: plugin.xml --- @@ -212,6 +212,27 @@ xmlns:android="http://schemas.android.com/apk/res/android"; +

[GitHub] cordova-plugin-file pull request: updated support for the OS X pla...

2014-04-30 Thread tripodsan
Github user tripodsan commented on the pull request: https://github.com/apache/cordova-plugin-file/pull/42#issuecomment-41845829 I think you should check our editor settings. it looks like the lines you changed have the wrong indentation. make sure you have "replace tabs with spaces"

[GitHub] cordova-plugin-file pull request: updated support for the OS X pla...

2014-04-30 Thread tripodsan
Github user tripodsan commented on a diff in the pull request: https://github.com/apache/cordova-plugin-file/pull/42#discussion_r12163593 --- Diff: src/osx/CDVFile.m --- @@ -0,0 +1,1024 @@ +/* + Licensed to the Apache Software Foundation (ASF) under one + or more contrib

[GitHub] cordova-plugin-file pull request: updated support for the OS X pla...

2014-04-30 Thread tripodsan
Github user tripodsan commented on a diff in the pull request: https://github.com/apache/cordova-plugin-file/pull/42#discussion_r12163564 --- Diff: src/osx/CDVLocalFilesystem.m --- @@ -0,0 +1,719 @@ +/* + Licensed to the Apache Software Foundation (ASF) under one + or mo

[GitHub] cordova-plugin-file pull request: updated support for the OS X pla...

2014-04-30 Thread tripodsan
Github user tripodsan commented on a diff in the pull request: https://github.com/apache/cordova-plugin-file/pull/42#discussion_r12163518 --- Diff: src/ios/CDVFile.m --- @@ -313,11 +313,9 @@ - (void)pluginInitialize --- End diff -- why is this changed? I thought y

[GitHub] cordova-plugin-file pull request: updated support for the OS X pla...

2014-04-30 Thread jsoref
Github user jsoref commented on a diff in the pull request: https://github.com/apache/cordova-plugin-file/pull/42#discussion_r12157924 --- Diff: doc/index.md --- @@ -69,6 +70,10 @@ Android also supports a special filesystem named "documents", which represents a By defaul

[GitHub] cordova-plugin-file pull request: updated support for the OS X pla...

2014-04-30 Thread jsoref
Github user jsoref commented on a diff in the pull request: https://github.com/apache/cordova-plugin-file/pull/42#discussion_r12157876 --- Diff: plugin.xml --- @@ -203,6 +203,27 @@ xmlns:android="http://schemas.android.com/apk/res/android"; +

[GitHub] cordova-plugin-file pull request: updated support for the OS X pla...

2014-04-30 Thread onflapp
GitHub user onflapp opened a pull request: https://github.com/apache/cordova-plugin-file/pull/42 updated support for the OS X platform - copying all the files to osx - making sure it works with the latest osx platform You can merge this pull request into a Git repository by runn