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

CB-8123 Plugin references can target specific windows platforms. #164

Closed
wants to merge 3 commits into
base: master
from

Conversation

Projects
None yet
4 participants
@TimBarham
Member

TimBarham commented Feb 11, 2015

Adds support for target, versions and arch attributes on <lib-file> and <framework> elements in the windows platform of plugin.xml. This allows plugin authors to target different references to different target platforms.

Also adds support for src attribute as an alias for the Include attribute on the <lib-file> element (since src is documented, but Include is used by existing plugins).

Adds some tests to cover the new attributes. Updates existing plugin tests for windows8 platform to also test windows platform (left in windows8 tests to help verify backward compatibility with old windows8 platform).

As part of this change, refactored jsproj to jsprojManager to reflect the fact that, with the windows platform, this class now manages multiple jsproj files.

Broken into three commits to separate file renames from changes, so that git can successfully track the changes.

TimBarham added some commits Feb 11, 2015

CB-8123 Plugin references can target specific windows platforms.
Adds support for `target`, `versions` and `arch` attributes on `<lib-file>` and `<framework>` elements in the windows platform of plugin.xml. This allows plugin authors to target different references to different target platforms.

Also adds support for `src` attribute as an alias for the `Include` attribute on the `
<lib-file>` element (since `src` is documented, but `Include` is used by existing plugins).

Adds some tests to cover the new attributes. Updates existing plugin tests for windows8 platform to also test windows platform (left in windows8 tests to help verify backward compatibility with old windows8 platform).

As part of this change, refactored `jsproj` to `jsprojManager` to reflect the fact that, with the windows platform, this class now manages multiple jsproj files.

I will rename some windows8 files and folders to windows, and jsproj.js to jsprojManager.js in a subsequent commit.
CB-8123 Rename windows platform related files.
Renames `windows8` plugin platform folders in tests to `windows`. Renames `jsproj.js` to `jsprojManager.js`.
CB-8123 Rename further windows platform related files.
Renames `windows8.spec.js` to `windows.spec.js`.
@sgrebnov

This comment has been minimized.

Show comment
Hide comment
@sgrebnov

sgrebnov Feb 11, 2015

Member

Reviewed and tested, LGTM 👍

Member

sgrebnov commented Feb 11, 2015

Reviewed and tested, LGTM 👍

@purplecabbage

This comment has been minimized.

Show comment
Hide comment
@purplecabbage

purplecabbage Feb 12, 2015

Contributor

Thanks Tim! This is now merged, the mirror is just slow to update. This should close in a few minutes.

Contributor

purplecabbage commented Feb 12, 2015

Thanks Tim! This is now merged, the mirror is just slow to update. This should close in a few minutes.

@TimBarham

This comment has been minimized.

Show comment
Hide comment
@TimBarham

TimBarham Feb 12, 2015

Member

Thanks Jesse!

Member

TimBarham commented Feb 12, 2015

Thanks Jesse!

@stefangordon

This comment has been minimized.

Show comment
Hide comment
@stefangordon

stefangordon Feb 12, 2015

Great stuff! Any pointers on how users can easily install this via NPM now? I've only had success manually cloning the lib and cli repos and copying correct binaries.

stefangordon commented Feb 12, 2015

Great stuff! Any pointers on how users can easily install this via NPM now? I've only had success manually cloning the lib and cli repos and copying correct binaries.

@TimBarham TimBarham closed this Feb 18, 2015

@TimBarham TimBarham deleted the MSOpenTech:CB-8123 branch Feb 18, 2015

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