Skip to main content
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index] [List Home]
Re: [cdt-dev] [tm-dev] [cross-project-issues-dev] TM Terminal and RSE releases for Photon (was: Correct repo for CDT to be referring to)
After more digging it seems that the TM.terminal bits built from master [1], published to the dev/nightly site [2], are built against Eclipse Mars [3], not Photon. 

I don't think this is wise, but I bet there's a technical reason (like guaranteeing backward compatibility for JDK 7?) so I'm not quite ready to just change this blindly. WDYT?

[2] http://download.eclipse.org/tm/terminal/builds/development/nightly/
[3] http://git.eclipse.org/c/tm/org.eclipse.tm.terminal.git/tree/admin/pom-config.xml#n73

But, if we want to use the Photon-built bits in the TM or CDT contribution to the Simrel, I've added a new deployment step to the 4.8-based job [4], which will push bits here [5].


Should we use the Photon-stack based build bits instead?

Also worthy of note:

tm.terminal depends on CDT. The default master build uses 8.8.1 [6]; the Photon profile uses 9.4 [7].

Is it bad to be shipping something destined to run with 9.x when we're building it against 8.x? Feels bad to me, but again, perhaps there's a reason for this ? 


Finally, I discovered that the tm.terminal build also depends on tm.rse [8], whose build in turn also depends on tm.terminal [9]. This circular build-time dependency is worrisome... and should be broken. I honestly don't know which project to build first in order.

We could merge the builds, move stuff around, etc. Again, WDYT?

[8] org.eclipse.tm.terminal.view.ui.rse depends on org.eclipse.rse.core, org.eclipse.rse.subsystems.files.core, org.eclipse.rse.ui
[9] org.eclipse.rse.terminals.ui depends on org.eclipse.tm.terminal.control



On Mon, Jun 11, 2018 at 11:39 AM, Doug Schaefer <dschaefer@xxxxxxxxxxxxxx> wrote:

+1. I trust you there. I don’t know the Terminal well enough to know whether the API changes in the deps would matter.

 

(BTW, removed cross-project as we’re getting a bit specific to TM).

 

From: cross-project-issues-dev-bounces@xxxxxxxxxxx [mailto:cross-project-issues-dev-bounces@xxxxxxxxxxx] On Behalf Of Nick Boldt
Sent: Monday, June 11, 2018 11:13 AM
To: TM project developer discussions <tm-dev@xxxxxxxxxxx>
Cc: Cross project issues <cross-project-issues-dev@eclipse.org>; CDT General developers list. <cdt-dev@xxxxxxxxxxx>
Subject: Re: [cross-project-issues-dev] [tm-dev] [cdt-dev] TM Terminal and RSE releases for Photon (was: Correct repo for CDT to be referring to)

 

Doug,

 

Do you agree we should respin TM against the latest Orbit and Photon deps? I'll assume so and try to find time today to get that built and pushed to the milestone site.

 

Nick

 

On Mon, Jun 11, 2018 at 10:16 AM, Doug Schaefer <dschaefer@xxxxxxxxxxxxxx> wrote:

CDT users use TM regularly, especially for Arduino and other embedded targets. Haven’t seen anything drastically broken so we haven’t done anything to it. I am a committer as well if a patch comes in. Other than that it’s in deep maintenance mode.

 

Doug.

 

From: cdt-dev-bounces@xxxxxxxxxxx [mailto:cdt-dev-bounces@eclipse.org] On Behalf Of Nick Boldt
Sent: Monday, June 11, 2018 9:25 AM
To: TM project developer discussions <tm-dev@xxxxxxxxxxx>
Cc: Cross project issues <cross-project-issues-dev@eclipse.org>; CDT General developers list. <cdt-dev@xxxxxxxxxxx>
Subject: Re: [cdt-dev] [tm-dev] TM Terminal and RSE releases for Photon (was: Correct repo for CDT to be referring to)

 

I stepped in recently to do releng for tm.rse & update it to build against Photon (because JBoss Tools requires it) but as yet I haven't done anything for tm.terminal, as I believe CDT Doug's been running the show there (because CDT requires it). 

 

If he's OK with me stepping in to help there too, I can find time this week to make sure there are new tm.terminal and tm.rse builds based on Photon.0.RC4. Can anyone on this list commit to smoke testing the builds? Other than "the tests pass in Jenkins" and "I can install everything from the update sites" I don't know how to verify the builds' contents are working as expected.

 

Nick 

 

On Mon, Jun 11, 2018 at 9:06 AM, Jonah Graham <jonah@xxxxxxxxxxxxxxxx> wrote:

Ni Nick,

 

Thanks for looking into this. I (on behalf of CDT) am not publishing the TM builds, simply consuming them when I noticed the discrepancy. Are you running releng for TM at the moment, or is this a continuing case of TM lacking the devs it needs?

 

AFAICT, the correct TM is being published, but from an unstable URL, so end users are getting the right version.

 

Jonah

 

 


~~~
Jonah Graham
Kichwa Coders Ltd.
www.kichwacoders.com

 

 

On Mon, 11 Jun 2018 at 13:49, Nick Boldt <nboldt@xxxxxxxxxx> wrote:

The nightly site contains something newer (2018/03/14) that the last milestone site (2018/03/12).

 

You can use p2 browser [1] to explore the sites [2] and pick an IU to investigate versions:

 

 

 

Given we're at RC4, it would probably be wise to release the 03/14 build to the milestone site. 

 

I can cobble a job together to do the publishing, unless you already have something you're using to publish updates. 

 

I notice the last tm.terminal build [3] was based on Photon.0.M6 so we should probably update that & rebuild it too to make sure it still works against Photon.RC4.

 

 

I suppose I should also do a TM.RSE release [4] this week too as the last build was back in March, also based on earlier Photon deps. 

 

 

 

 

On Sat, Jun 9, 2018 at 6:13 PM, Jonah Graham <jonah@xxxxxxxxxxxxxxxx> wrote:

Hi TM folk,

 

I am doing a review of CDT's target platform for the 9.5 branch (in preparation for Photon release) and I came across a minor anomaly I wanted to resolve.

 

At the moment CDT uses a build from 2 days before what TM is contributing in the simrel.

 

 

I don't really want to have CDT point at a development/nightly build on the 9.5 branch. Is there any difference between the two. Does TM plan to publish a URL suitable for building against Photon?

 

Thanks,

Jonah


~~~
Jonah Graham
Kichwa Coders Ltd.
www.kichwacoders.com


_______________________________________________
tm-dev mailing list
tm-dev@xxxxxxxxxxx
To change your delivery options, retrieve your password, or unsubscribe from this list, visit
https://dev.eclipse.org/mailman/listinfo/tm-dev



 

--

Nick Boldt

Principal Software Engineer, RHCSA

Productization Lead :: JBoss Tools & Dev Studio

IM: @nickboldt / @nboldt / http://nick.divbyzero.com

 

 

“The Only Thing That Is Constant Is Change” - Heraclitus

_______________________________________________
tm-dev mailing list
tm-dev@xxxxxxxxxxx
To change your delivery options, retrieve your password, or unsubscribe from this list, visit
https://dev.eclipse.org/mailman/listinfo/tm-dev


_______________________________________________
tm-dev mailing list
tm-dev@xxxxxxxxxxx
To change your delivery options, retrieve your password, or unsubscribe from this list, visit
https://dev.eclipse.org/mailman/listinfo/tm-dev



 

--

Nick Boldt

Principal Software Engineer, RHCSA

Productization Lead :: JBoss Tools & Dev Studio

IM: @nickboldt / @nboldt / http://nick.divbyzero.com

 

 

“The Only Thing That Is Constant Is Change” - Heraclitus


_______________________________________________
tm-dev mailing list
tm-dev@xxxxxxxxxxx
To change your delivery options, retrieve your password, or unsubscribe from this list, visit
https://dev.eclipse.org/mailman/listinfo/tm-dev



 

--

Nick Boldt

Principal Software Engineer, RHCSA

Productization Lead :: JBoss Tools & Dev Studio

IM: @nickboldt / @nboldt / http://nick.divbyzero.com

 

 

“The Only Thing That Is Constant Is Change” - Heraclitus


_______________________________________________
tm-dev mailing list
tm-dev@xxxxxxxxxxx
To change your delivery options, retrieve your password, or unsubscribe from this list, visit
https://dev.eclipse.org/mailman/listinfo/tm-dev



--

Nick Boldt

Principal Software Engineer, RHCSA

Productization Lead :: JBoss Tools & Dev Studio

IM: @nickboldt / @nboldt / http://nick.divbyzero.com



“The Only Thing That Is Constant Is Change” - Heraclitus

Back to the top