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

Please merge 37014, 37125, 37128, and 37204 to trunk #19319

Closed
johnmccutchan opened this issue Jun 9, 2014 · 8 comments
Closed

Please merge 37014, 37125, 37128, and 37204 to trunk #19319

johnmccutchan opened this issue Jun 9, 2014 · 8 comments
Assignees
Labels
area-vm Use area-vm for VM related issues, including code coverage, FFI, and the AOT and JIT backends. merge-to-dev type-task
Milestone

Comments

@johnmccutchan
Copy link
Contributor

37125 & 37128 contain the final set of fixes for Observatory 1.5.

@johnmccutchan
Copy link
Contributor Author

Please hold off on the merge of 37128. Justin just informed me there is a new version of polymer_expressions which I should use.

@kasperl
Copy link

kasperl commented Jun 10, 2014

So this is a request to just merge 37125? Treating as such.


Removed this from the milestone.
Added this to the 1.5 milestone.
Added NeedsInfo label.
Changed the title to: "Please merge 37125".

@kasperl
Copy link

kasperl commented Jun 10, 2014

Added New label.
Changed the title to: "Please merge 37125 to trunk".

@ricowind
Copy link
Contributor

This does not merge cleanly:
ricow@nobbo:~/merge$ svn merge -c 37125 https://dart.googlecode.com/svn/branches/bleeding_edge trunk

Conflict discovered in 'trunk/dart/runtime/bin/vmservice/client/lib/src/app/location_manager.dart'.

--- Merging r37125 into 'trunk/dart':
C trunk/dart/runtime/bin/vmservice/client/lib/src/app/location_manager.dart
Conflict discovered in 'trunk/dart/runtime/bin/vmservice/client/lib/src/app/application.dart'.
C trunk/dart/runtime/bin/vmservice/client/lib/src/app/application.dart
Conflict discovered in 'trunk/dart/runtime/bin/vmservice/client/lib/src/elements/nav_bar.html'.
C trunk/dart/runtime/bin/vmservice/client/lib/src/elements/nav_bar.html
U trunk/dart/runtime/bin/vmservice/client/lib/src/elements/isolate_summary.html
U trunk/dart/runtime/bin/vmservice/client/lib/service_html.dart


Set owner to @johnmccutchan.
Added NeedsInfo label.

@johnmccutchan
Copy link
Contributor Author

Attached is a patch against trunk that updates the deployed version of Observatory for 1.5.


Attachment:
observatory15.patch (1.14 MB)

@ricowind
Copy link
Contributor

The supplied patch is more more than just 37125 and 37128 - it is pretty hard to provide good log entries in the 1.5 commit log when I don't know what I am effectively merging (I am taking this over anyway, please update here anyway so we can track it)

@johnmccutchan
Copy link
Contributor Author

Hi,

Sorry for leaving that information out, the attached patch is: 37014, 37125, 37128, and 37204 but only the changes in deployed directory, not the dart sources used only for development.

The reason my original request (37125 & 37128) didn't apply cleanly was because I was under the impression 37104 made it into the final full push to trunk.

John

@kasperl
Copy link

kasperl commented Jun 11, 2014

Fixed in r37221 (1.5.0-dev.4.6).

I wonder if it's really a good idea to leave out the development Dart files. It seems like that could cause interesting conflicts in the future when merging additional fixes over. Basically, all future fixes for the Observatory now needs more manual work which is a bad thing.


Added Fixed label.
Changed the title to: "Please merge 37014, 37125, 37128, and 37204 to trunk".

@johnmccutchan johnmccutchan added merge-to-dev area-vm Use area-vm for VM related issues, including code coverage, FFI, and the AOT and JIT backends. type-task Fixed labels Jun 11, 2014
@johnmccutchan johnmccutchan self-assigned this Jun 11, 2014
@johnmccutchan johnmccutchan added this to the 1.5 milestone Jun 11, 2014
This issue was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area-vm Use area-vm for VM related issues, including code coverage, FFI, and the AOT and JIT backends. merge-to-dev type-task
Projects
None yet
Development

No branches or pull requests

3 participants