Skip to content

Allow dependencies to be installed inside a .sublime-package#1291

Closed
evandrocoan wants to merge 1 commit intosublimehq:masterfrom
evandrocoan:on_sublime_package_feature
Closed

Allow dependencies to be installed inside a .sublime-package#1291
evandrocoan wants to merge 1 commit intosublimehq:masterfrom
evandrocoan:on_sublime_package_feature

Conversation

@evandrocoan
Copy link
Copy Markdown

My dependency can be packed inside a .sublime-package, there no
need to it be on the loose user packages folder. When I need it, I
just import like:

from DependencyName.file_name import CoolClass

Then if I create the file on.sublime-package on the root of my directory, the dependency will be packaged inside a .sublime-package, instead of the default unpacked.

My dependency can be packed inside a `.sublime-package`, there no
need to it be on the loose user packages folder. When I need it, I
just import like:
```python
from DependencyName.file_name import CoolClass
```
@FichteFoll
Copy link
Copy Markdown
Contributor

This isn't needed. Usually, Python packages expect to be in an unextracted way and there is no downside in having PC unextract them when installing. The only reason I could see for this is when the dependencies clutter the packages folder, in which case #800 is the proper fix.

@evandrocoan
Copy link
Copy Markdown
Author

Moving then to a different folder as lib/python3.3 would also help, as long as I can still call they like from DependencyName.file_name import CoolClass.

@evandrocoan evandrocoan closed this Nov 2, 2017
@evandrocoan evandrocoan deleted the on_sublime_package_feature branch November 2, 2017 15:02
evandrocoan added a commit to evandrocoan/EstimatedTimeLeft that referenced this pull request Dec 9, 2017
Allow dependencies to be installed inside a .sublime-package
sublimehq/package_control#1291
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