Re: [linuxtools-dev] Contributing to the Docker tools

[Aleksandar Kurtakov <akurtako@xxxxxxxxxx>]

----- Original Message -----
> From: "Mario" <mario.loriedo@xxxxxxxxx>
> To: "Linux Tools developer discussions" <linuxtools-dev@xxxxxxxxxxx>
> Sent: Friday, 6 November, 2015 1:19:14 PM
> Subject: Re: [linuxtools-dev] Contributing to the Docker tools
> 
> Hello Max,
> 
> I was looking for a superman t-shirt during the talk but could not spot you
> ;-P
> 
> By the way happy to contribute. I will follow the link you provided to accept
> the CLA.
> 
> I've got one question : once the code is merged would it be possible to add
> my company (Zenika https://zenika.com/ ) to the active member companies list
> ?

Two answers:
1. If by active member you mean eclipse foundation membership - this is something that the project can't handle but it's between Eclipse foundation and your company (see https://eclipse.org/membership/become_a_member/). If/when you settle this your company will show on every project where there is commit from someone@xxxxxxxxxxxxxxx. - Well established process but not in our powers.
2. If you want to have something like "Dockerfile editor support provided initially by Zenika." there is nothing stopping us from adding it to the docker page (https://www.eclipse.org/linuxtools/projectPages/docker/). - Not done in general but should be a no brainer and something we can do in few minutes.

Alexander Kurtakov
Red Hat Eclipse team



> That's not blocker but it will be definitely easier for me to continue
> maintaining the code in the future.
> 
> Mario
> 
> On Thu, Nov 5, 2015 at 9:18 PM, Max Rydahl Andersen < manderse@xxxxxxxxxx >
> wrote:
> 
> 
> Hi Mario,
> 
> Glad you found Alexander at EclipseCon
> - I did see part of your talk but I didn't manage to
> locate you afterwards.
> 
> 
> 
> 
> 
> Basically doclipser [1] is an Xtext plugin for Dockerfiles grammar with some
> extra code do add items on the "Run as" menu to execute Docker commands.
> There is almost no overlap with the Docker tools you wrote so it makes sense
> to merge it all together.
> 
> This sounds great! We've wanted to improve the user experience for editing
> Dockerfiles for some time. One entry point would be in our ImageBuildPage
> where a very basic dialog is opened with a basic text area for editing.
> 
> 
> 
> In addition from the context of the editor, your contribution could take
> advantage of the API we have for building images/running containers.
> 
> Why should the editor do that ? isn't that just context menu on the file ?
> 
> 
> 
> 
> 
> I will look at the linux-tools code in the next days to understand what's the
> best way to start and will be back to you with a proposal.
> 
> In the meantime please let me know if you have any concerts or suggestions.
> 
> Observations from just a quick look over the code :
> 
> - I See you're using spotify/docker-client as your main client but also have
> some support for dockerjava. We ended up going with spotify/docker-client so
> if all functionality is supported with this, I think integrating should be
> very easy
> 
> - You seem to be bundling (Bundle-ClassPath) docker-client into
> com.zenika.doclipser.api to make it available but ideally we'd have to
> provide
> it as a proper OSGi bundle like everything else. Maybe you looked at the
> various
> dependencies it has and realized it'd be a lot to provide a manifest for
> everything. We've actually gone through this process for docker-client at
> Eclipse Orbit, so you would be able to simply do something like
> 'Require-Bundle:
> com.spotify.docker.client' and it would work as expected.
> 
> - Dependencies of the project seem sane so I don't see us filing any
> additional CQs if we merge the codebase
> 
> Sounds good - I thought we had a concern about depending on xtext earlier,
> but that has changed or ?
> 
> 
> 
> - The project builds with Tycho, so that would make integrating with our
> build simpler
> 
> - The code seems to be entirely done mainly by you so the legal process
> should
> be simpler (also because it's EPL), although you might need to sign the
> Eclipse CLA (not entirely sure here)
> 
> With contribution of this size he must/should.
> 
> Sign in to
> https://dev.eclipse.org/site_login/?takemeback=https%3A//projects.eclipse.org/user/login/sso/pmi
> and accept the CLA. Done ;)
> 
> /max
> http://about.me/maxandersen
> 
> _______________________________________________
> linuxtools-dev mailing list
> linuxtools-dev@xxxxxxxxxxx
> To change your delivery options, retrieve your password, or unsubscribe from
> this list, visit
> https://dev.eclipse.org/mailman/listinfo/linuxtools-dev
> 
> 
> _______________________________________________
> linuxtools-dev mailing list
> linuxtools-dev@xxxxxxxxxxx
> To change your delivery options, retrieve your password, or unsubscribe from
> this list, visit
> https://dev.eclipse.org/mailman/listinfo/linuxtools-dev