[ALOY-1485] Warning on relocation of platform folder confusing if it contains only one platform
GitHub Issue | n/a |
---|---|
Type | Improvement |
Priority | None |
Status | Open |
Resolution | Unresolved |
Affected Version/s | alloy 1.8.0, alloy 1.8.5 |
Fix Version/s | n/a |
Components | Tooling |
Labels | n/a |
Reporter | Fokke Zandbergen |
Assignee | Chris Barber |
Created | 2016-04-01T14:00:56.000+0000 |
Updated | 2016-04-01T14:05:41.000+0000 |
Description
Take the following scenario:
I have an Alloy app that targets iOS and Android but only has a
I continue to work on the iOS version of my app and get a warning to add
why would I stop versioning that if it contains important stuff for Android?
I have an Alloy app that targets iOS and Android but only has a /platform/android
, no /platform/ios
.
I upgraden from Alloy 1.7.35 to 1.8.0, which includes ALOY-1365.
I continue to work on the iOS version of my app and get a warning to add /platform
to my .gitignore
file. Weird, because
why would I stop versioning that if it contains important stuff for Android?
After a while I switch to work on the Android version and it says:
[ERROR] Detected legacy "/platform" directory in project directory.
[ERROR] Please move the "/platform" directory to "/app/platform" for Alloy 1.8.0 or later.
[ERROR] Alloy compiler failed
This is confusing because didn't I work on my iOS app successfully using Alloy 1.8.0 without having moved the folder?
Also, when I'd now move /i18n
to /app/i18n
as it suggests it would now also include an ios
folder which contains a alloy_generated
file.
I think it would be better to do the check (and have the alloy_generated
file in the root of /platform
and not in the platform folders. It would then:
* Already have failed to build when I started working on IOS after the upgrade.
* The warning would not be confusing since it would only show on the second build after dealing with...
* The error would not be confusing since it is about the root folder and not the platform one.
* The /platform/ios
would not have been generated, easing the relocation.
No comments