Sounds good. That particular functionality may no longer be
needed. Some of the ADT functionality could also be deprecated
in favor of code that came from the old Motrola Studio device
views as well.
Dave
I have pressed on,
patching the code to avoid compilation errors, and one issue
needs to be resolved from this activity.
There is a class named
DeviceMenuListener in "adt" package
org.eclipse.andmore.internal.editors.layout.configuration
which has the following logic:
"Group the devices by
manufacturer, then put them in the menu. If we don't have
anything but Nexus devices, group them together rather than
make many manufacturer submenus."
I had to comment out
the Nexus code as it references objects in an sdk-common
helper class which have been removed. Maybe a bug report
needs to be raised so the design can be reviewed.
I am now at the stage
of having set up a test environment to proove the PR. I am
hoping there are no big surprizes so I can submit the PR
shortly.
Andrew
Andrew, thanks. Can you put a pull request together that
contains all the fixes, and I'll try to get it going through the
simulataneous review process. This will allow us to merge the
change, while allowing the IP team to do their review.
In this case, it would be better to have one commit with all
the changes, so it would be easier to back out if we need to for
any reason. If you follow the format of the other commit
messages, the PR will automatically be linked back to bugzilla.
Thanks for the all the effort on this.
Dave
On 10/15/2017 4:50 PM, Andrew Bowley
wrote:
Please find attached file
containing information on dependency jars for the CQ
process.
The bug fix requires Andriod
SDK builder-2.3.3.jar which declares 34 dependencies.
These include 7 other SDK artifacts which I indicate are
not expected to be deployed at this time.
Andrew
_______________________________________________
andmore-dev mailing list
andmore-dev@xxxxxxxxxxx
To change your delivery options, retrieve your password, or unsubscribe from this list, visit
https://dev.eclipse.org/mailman/listinfo/andmore-dev
_______________________________________________
andmore-dev mailing list
andmore-dev@xxxxxxxxxxx
To change your delivery options, retrieve your password, or unsubscribe from this list, visit
https://dev.eclipse.org/mailman/listinfo/andmore-dev
