Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Drop support for linking (§8) #26

Closed
raphinesse opened this issue Sep 10, 2018 · 0 comments · Fixed by #40
Closed

Drop support for linking (§8) #26

raphinesse opened this issue Sep 10, 2018 · 0 comments · Fixed by #40
Assignees

Comments

@raphinesse
Copy link
Contributor

raphinesse commented Sep 10, 2018

From @raphinesse on July 5, 2018 9:4

Current situation

Ignoring all special cases discussed in #103, cordovaCreate w/ enabled link option will

  • symlink folders www, merges and hooks
  • symlink the file config.xml

Pain Points

Proposal

Drop it. 🔥

It seems it was planned to remove --link-to before apache/cordova-discuss#49 (comment). Support was then removed in apache/cordova-lib#456 and re-added in 79cace5 addressing https://issues.apache.org/jira/browse/CB-11623 which is still open because the current implementation apparently still not satisfies the reporter's requirements. 😒

The reporter's requirements are to create a cordova project with www linking to some existing web app. This could be easily achieved by running

$ cordova create bin com.example.domain APPNAME
$ rm -r bin/www && ln -sr www bin/www

instead of

$ cordova create bin com.example.domain APPNAME --link-to=www

Evolved from apache/cordova-discuss#89

Copied from original issue: apache/cordova-discuss#104

@raphinesse raphinesse self-assigned this Sep 10, 2018
@raphinesse raphinesse changed the title [cordova-create] Drop support for linking (§8) Drop support for linking (§8) Sep 10, 2018
raphinesse added a commit to raphinesse/cordova-create that referenced this issue Oct 15, 2019
raphinesse added a commit to raphinesse/cordova-create that referenced this issue Oct 16, 2019
@erisu erisu closed this as completed in #40 Nov 5, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant