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

chore: shift ios project to this project #606

Merged
merged 1 commit into from
Jan 3, 2020

Conversation

adityastic
Copy link
Collaborator

Fixes #604

@auto-label auto-label bot added the chore label Jan 3, 2020
@adityastic adityastic changed the title chore: shift ios project to this project [WIP] chore: shift ios project to this project Jan 3, 2020
@auto-label auto-label bot removed the chore label Jan 3, 2020
@adityastic adityastic force-pushed the patch-604 branch 3 times, most recently from 83ff844 to 69bb2f9 Compare January 3, 2020 15:28
@adityastic adityastic changed the title [WIP] chore: shift ios project to this project chore: shift ios project to this project Jan 3, 2020
@adityastic adityastic removed the WIP label Jan 3, 2020
@adityastic adityastic force-pushed the patch-604 branch 12 times, most recently from c43c956 to 8bccef8 Compare January 3, 2020 17:14
@adityastic
Copy link
Collaborator Author

@iamareebjamal travis is one hell of a mess. It does not allow you to use android language on macOS image.
Timeline:

  1. Thought of running gradle on macOS by downgrading the jdk to 8.
  2. It was still building the android project even when I was building a specific module with
    ./gradlew BadgeMagicModule:packForXcode
  3. I finally managed to make it work by not including the android project in gradle.settings
  4. So, I combined it all and removed the android project on macOS push.

Let me know your thoughts on this approach. This is a workaround and I'm sure there must be a better way or we can switch to Github Workflow which might be better as a whole.

@adityastic
Copy link
Collaborator Author

@iamareebjamal if we can manage to build only the module we can solve this without issues, but for some reason even after specifying the module with the command it builds the android project which fails for some unknown reason https://travis-ci.com/fossasia/badge-magic-android/jobs/272179208.

# -----------
# -----------
# Remove Project from gradle.settings
- echo "include ':BadgeMagicModule'" > settings.gradle
Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@iamareebjamal that's the hack for now

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Works for now

@adityastic adityastic changed the title chore: shift ios project to this project [WIP] chore: shift ios project to this project Jan 3, 2020
@auto-label auto-label bot removed the chore label Jan 3, 2020
@iamareebjamal
Copy link
Member

Merge after changing the PR title

@iamareebjamal
Copy link
Member

iamareebjamal commented Jan 3, 2020

The build is failing because gradle is built on JDK 12 and JDK 13 is installed on the platform. gradle only supports JDK <= 12

So, if you can use that, it'll fix the error. But for now, this is fine. Open an issue to improve the implementation later on

@adityastic
Copy link
Collaborator Author

@iamareebjamal Yes, Even I thought the same and tried all jdk's from 8 to 12, all failed because of Kotlin NullPointerException in Databinding. Here are the links of all builds, It should not build the android project as I was specifying the module specifically, but for some reason it was.

jdk8 - https://travis-ci.com/fossasia/badge-magic-android/jobs/272161844
jdk9 - https://travis-ci.com/fossasia/badge-magic-android/jobs/272180978
jdk11 - https://travis-ci.com/fossasia/badge-magic-android/jobs/272181067
jdk12 - https://travis-ci.com/fossasia/badge-magic-android/jobs/272181305

@adityastic adityastic merged commit 0148c45 into fossasia:development Jan 3, 2020
@adityastic adityastic changed the title [WIP] chore: shift ios project to this project chore: shift ios project to this project Jan 3, 2020
@adityastic adityastic deleted the patch-604 branch January 4, 2020 00:25
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.

shifting iOS app here
2 participants