Titanium JIRA Archive
Titanium SDK/CLI (TIMOB)

[TIMOB-17091] Core: Remove binary files from repository

GitHub Issuen/a
TypeEpic
PriorityHigh
StatusClosed
ResolutionWon't Fix
Resolution Date2017-07-26T03:55:43.000+0000
Affected Version/sRelease 3.2.3
Fix Version/sn/a
ComponentsiOS
Labelsn/a
ReporterMatt Langston
AssigneeEric Merriman
Created2014-06-09T15:42:23.000+0000
Updated2017-07-26T03:55:43.000+0000

Description

The titanium_mobile github repo should contain a minimum of binary files in order to decrease the size of the repository. Here is the current list:

.a Files

/demos/ApiCheck/build/iphone/Resources/libTitanium.a /iphone/Classes/APSAnalytics/libAPSAnalytics.a /iphone/Classes/APSHTTPClient/libAPSHTTPClient.a /iphone/lib/libti_ios_debugger.a /iphone/lib/libti_ios_profiler.a /iphone/lib/libtiverify.a

Modules

/support/module/packaged/com.appcelerator.urlSession-iphone-1.0.1.zip /support/module/packaged/facebook-android-3.0.2.zip /support/module/packaged/facebook-iphone-3.1.1.zip /support/module/packaged/ti.cloud-commonjs-3.2.3.zip /support/module/packaged/ti.cloudpush-android-3.3.0.zip /support/module/packaged/ti.map-android-2.1.4.zip /support/module/packaged/ti.map-iphone-2.0.2.zip

.jar Files

/android/build/lib/ant-contrib-1.0b3.jar /android/build/lib/ant-launcher.jar /android/build/lib/ant-nodeps.jar /android/build/lib/ant.jar /android/build/lib/json_simple-1.1.jar /android/build/lib/publishedApiDoclet1.5-0.4.jar /android/build/lib/serialver.jar /android/build/lib/xercesImpl.jar /android/build/lib/xml-apis.jar /android/kroll-apt/lib/freemarker.jar /android/kroll-apt/lib/json_simple-1.1.jar /android/modules/appcompat/lib/android-support-v4.jar /android/modules/appcompat/lib/android-support-v7-appcompat.jar /android/modules/xml/lib/jaxen-1.1.1.jar /android/titanium/lib/aps-analytics.jar /android/titanium/lib/commons-logging-1.1.1.jar /android/titanium/lib/nineoldandroids-appc-2.4.0.jar /android/titanium/lib/thirdparty.jar /android/titanium/lib/ti-commons-codec-1.3.jar /demos/ApiCheck/build/android/lib/titanium.jar /node_modules/node-appc/tools/resizer/lib/imgscalr-lib-4.2.jar /node_modules/node-appc/tools/resizer/resizer.jar /support/android/lib/closure-compiler.jar /support/android/lib/titanium-debug.jar /support/android/lib/titanium-profiler.jar /support/android/lib/titanium-verify.jar /support/mobileweb/closureCompiler/compiler.jar /support/mobileweb/imageResizer/imgscalr-lib-4.2.jar

Comments

  1. Ingo Muschenetz 2014-06-09

    Why would there be merge conflicts in the .zip files? Have you updated your versions?
  2. Matt Langston 2014-06-09

    No, I have not updated my versions. However, binary files should not be stored in github to begin with. For example, un-versioniond binary libraries stored in github makes it more difficult to maintain our software and makes it more difficult for customers to use our software. I advocate we investigate Cocoa Pods for the dependency management of our Xcode projects. If not Cocoa Pods then a simple Xcode workspace (a simple text file) will suffice.
  3. Ingo Muschenetz 2014-06-12

    Okay. I was just curious why you would get merge conflicts on these files. Regardless, it looks like we'll move to cocoapods sooner over later.
  4. Matt Langston 2014-06-20

    More merge conflicts. SDK Team, please fix this ASAP. This slows me down testing PRs and contributing to the project. titanium_mobile (master)$ git merge appcelerator/master warning: Cannot merge binary files: support/module/packaged/ti.cloud-commonjs-3.2.3.zip (HEAD vs. appcelerator/master) warning: Cannot merge binary files: iphone/Classes/APSAnalytics/libAPSAnalytics.a (HEAD vs. appcelerator/master) warning: Cannot merge binary files: android/titanium/lib/aps-analytics.jar (HEAD vs. appcelerator/master) Removing support/module/packaged/ti.cloudpush-android-3.2.1.zip Auto-merging support/module/packaged/ti.cloud-commonjs-3.2.3.zip CONFLICT (content): Merge conflict in support/module/packaged/ti.cloud-commonjs-3.2.3.zip Auto-merging support/module/packaged/com.appcelerator.urlSession-iphone-1.0.1.zip Auto-merging iphone/iphone/Titanium.xcodeproj/project.pbxproj Auto-merging iphone/cli/commands/_build.js Auto-merging iphone/Classes/TiNetworkHTTPClientProxy.m Removing iphone/Classes/APSHTTPClient/APSHTTPOperation.h Auto-merging iphone/Classes/APSAnalytics/libAPSAnalytics.a CONFLICT (content): Merge conflict in iphone/Classes/APSAnalytics/libAPSAnalytics.a Auto-merging apidoc/Titanium/Analytics/Analytics.yml Auto-merging android/titanium/src/java/org/appcelerator/titanium/view/TiUIView.java Auto-merging android/titanium/src/java/org/appcelerator/titanium/util/TiAnimationBuilder.java Auto-merging android/titanium/lib/aps-analytics.jar CONFLICT (content): Merge conflict in android/titanium/lib/aps-analytics.jar Auto-merging android/modules/ui/src/java/ti/modules/titanium/ui/widget/listview/ListSectionProxy.java Auto-merging android/modules/network/src/java/ti/modules/titanium/network/TiHTTPClient.java Auto-merging android/modules/analytics/src/java/ti/modules/titanium/analytics/AnalyticsModule.java CONFLICT (content): Merge conflict in android/modules/analytics/src/java/ti/modules/titanium/analytics/AnalyticsModule.java Automatic merge failed; fix conflicts and then commit the result.
  5. Ingo Muschenetz 2014-06-22

    I've listed what I believe are the current binary files in the ticket. Let's create tickets for each of these. I don't know if we need to remove all binary files, or just the ones that change frequently.
  6. Ingo Muschenetz 2014-06-22

    I believe we may wish to move all dependency management to something like Gradle, since a number of our dependencies are _not_ open-sourced. See http://www.gradle.org/docs/current/userguide/dependency_management.html for more information.
  7. Matt Langston 2014-06-22

    Great inventory of our binary files [~ingo] - than you!
  8. Chris Barber 2016-09-20

    [~ingo] I'm not sure why this was ever a good idea. Removing these files now won't make the repo smaller. Besides, it's not like these files are huge. Having scons download them will only complicate things. We will have to manually version files and you will need an Internet connection if you are missing any of these files. I recommend we resolve this ticket as won't fix.
  9. Lee Morris 2017-07-26

    Resolving ticket after advice from above.

JSON Source