Activity

1. pif commented on Mar 30, 2013

   

   pif

   on Mar 30, 2013

   More actions

   I managed to build & run android sketches from command line with ./processing-java. I had to change some lines to achieve that (~250-270 in Commander.java and Devices/AndroidBuild classes visibility).
   Something like:

         AndroidMode androidMode = (AndroidMode) ModeContribution.load(null, Base.getContentFile("modes/android"),
             "processing.mode.android.AndroidMode").getMode();      
         androidMode.checkSDK(null);
   
         try {
           sketch = new Sketch(pdePath, androidMode);
           if (task == BUILD || task == RUN || task == PRESENT) {
             AndroidBuild build = new AndroidBuild(sketch, androidMode);
             build.build("debug");
             AndroidRunner runner = new AndroidRunner(build, this);
             runner.launch(Devices.getInstance().getHardware());
   

   However, how should command-line evolve for android?

   1. as a standalone shell script ./processing-android and class AndroidCommander;
   2. make ./processing-java more general: add --mode=[java/android/etc.], decouple Commander from Java mode?
2. benfry commented on Apr 11, 2013

   

   benfry

   on Apr 11, 2013

   Contributor

   More actions

   Yeah, it's not a difficult change, so much as it just needs some babysitting.

   I decided that the command line versions should live in the individual modes, since they behave somewhat differently and start diverging fairly quickly. So to answer your question, a separate Commander class over in processing.mode.android would be the right way to go.
3. pif

   added 5 commits that reference this issue on Apr 13, 2013

   fixed processing#1361: added android command-line support

   This issue will close once commit 3957064 is merged into the 'master' branch.

   3957064

   fixed processing#1361 android command-line

   This issue will close once commit 879424e is merged into the 'master' branch.

   879424e

   fixed processing#1361 removed debug info

   This issue will close once commit 3c74848 is merged into the 'master' branch.

   3c74848

   fixed processing#1361 updated sources

   This issue will close once commit c572cf2 is merged into the 'master' branch.

   c572cf2

   issue processing#1361: removed general cli files

   This issue will close once commit bddbea5 is merged into the 'master' branch.

   bddbea5
4. processing-bugs commented on May 14, 2013

   

   processing-bugs

   on May 14, 2013

   Author

   More actions

   Available in the next release. Thanks again!
5. processing-bugs

   closed this as completedon May 14, 2013
6. github-actions commented on Jun 17, 2021

   

   github-actionsBot

   on Jun 17, 2021

   More actions

   This issue has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.
7. github-actions

   locked as resolved and limited conversation to collaborators on Jun 17, 2021