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

Added support for specifying packages to import in library.properties (updated version of PR #2178) #3010

Merged
merged 8 commits into from Jan 19, 2015

Conversation

Projects
None yet
3 participants
@joelmoniz
Member

joelmoniz commented Dec 17, 2014

This is the updated, auto-merge-able version of PR #2178. It closes PR #2178 and fixes #109.

(The following is taken from the original PR, #2172:)


Added support for specifying packages to import in library.properties. Searches for 'imports' as a key in library.properties.

imports=mylib, mylib.util

If found, these imports are added when the user selects 'Import Library'.

import mylib.*
import mylib.util.*

If this line doesn't exist, falls back to current behavior and adds imports for all packages in the library's base jar.

This involved one substantive change: passing the name of the library to JavaEditor.handleImportLibrary(String) rather than the jar path. Once the library is found (by name), then its jar path is available via Library.getJarPath().

benfry added a commit that referenced this pull request Jan 19, 2015

Merge pull request #3010 from joelmoniz/libProps
Added support for specifying packages to import in library.properties (updated version of PR #2178)

@benfry benfry merged commit b2130bf into processing:master Jan 19, 2015

@joelmoniz joelmoniz deleted the joelmoniz:libProps branch Feb 19, 2015

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment