Skip to main content
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index] [List Home]
Re: [mylyn-docs-dev] exploring changes to Mylyn Docs build
Before responding, I want to point out that I'm not a Mylyn Docs committer. I'm just offering my thoughts for consideration.

I don't feel strongly about this decision but I do question some of the claimed benefits:
 
* easier to contribute (e.g. without knowledge of OSGi or PDE)

I would argue that most contributions don't actually require any knowledge of OSGi or PDE. For larger contributions, the knowledge required is usually just how to add dependencies to a manifest, which shouldn't be an obstacle for someone invested enough to make a large contribution.

* better structured code/tests (e.g. using normal Maven project layout)

I don't think that following a different convention is "better," just different.
 
* faster cycle time (e.g. easier to verify code changes, faster builds)

Is there any data to support this? This should be weighed against the increased friction of needing to run a maven build when debugging. It seems to me that this will affect all development on WikiText, not just UI development.

Is there a way to have the best of both worlds by enabling the core to be built using either Maven or PDE? This would probably add a little maintenance cost but the smoother development workflow might be worth it.

Cheers,
Sam


--
Sam Davis
Senior Software Engineer, Tasktop
Committer, Eclipse Mylyn
http://tasktop.com

On Tue, Jan 24, 2017 at 4:55 PM, David Green <david.green@xxxxxxxxxxx> wrote:
All,

The build change experiment is coming together and I want to be sure that I close the loop before proceeding.  You can see what I've done so far here: 


There is now a single-command build as detailed in the readme.

See the discussion below on this thread for detail about trade-offs. I think it's worth moving ahead with this knowing that the UI development workflow is not perfect, but your opinion may differ so please let me know if you think we should hold off.

David


On Mon, Dec 5, 2016 at 11:40 AM David Green <david.green@xxxxxxxxxxx> wrote:
Stephan,

With the proposed changes as-is, that would not be possible.  You'd have to run a maven build on the core bundles and then refresh the projects and/or target platform to consume the changes.  This workflow is definitely suboptimal.

If we make it easier to write unit tests and run them, it will become a lot easier to debug core portions of the codebase without having to fire up an IDE from your workspace.  At least that's my hope with the Maven project layout.

I've  found that we're spending less time working on RCP/UI portions of the codebase and more on the core portions.  
At least for me, this would be an acceptable trade-off.  What do you think?

David

On Sat, Dec 3, 2016 at 1:19 PM Stephan Wahlbrink <sw@xxxxxxxxxxxx> wrote:
Hi David,

[2016-12-02 18:47] David Green wrote:
> Devs,
>
> I've been experimenting with restructuring the Mylyn Docs build to
> achieve a few things:
>
> * easier to build/release Mylyn Docs (or at least Mylyn WikiText)
> separately from Mylyn
> * easier to consume Mylyn WikiText core bundles (e.g. as plain jars,
> with normal poms)
> * easier to contribute (e.g. without knowledge of OSGi or PDE)
> * better structured code/tests (e.g. using normal Maven project layout)
> * faster cycle time (e.g. easier to verify code changes, faster builds)
>
> To achieve these things I've been looking at building the "core"
> wikitext bundles as normal Maven artifacts, using a Maven plug-in to
> generate OSGi manifests.

How can I consume such m2e source projects directly as osgi bundles in a
PDE workspace / PDE build? (use case: debugging Mylyn WikiText core
bundles in an RCP application)

Stephan


> You can see the results of my experiment here:
> https://github.com/greensopinion/mylyn.docs
>
> The build is split into two parts: a "core" part which is Tycho-less,
> and a UI/OSGi part which uses Tycho.
>
> I'd love to get your thoughts and feedback on the goals and the approach.
>
> David
>
>
> --
>
> *
> *
>
> *David Green **|** *VP of Architecture* **| *Tasktop
>
> *email: *david.green@xxxxxxxxxxx <mailto:david.green@tasktop.com>
>
>
>
> _______________________________________________
> mylyn-docs-dev mailing list
> mylyn-docs-dev@xxxxxxxxxxx
> To change your delivery options, retrieve your password, or unsubscribe from this list, visit
> https://dev.eclipse.org/mailman/listinfo/mylyn-docs-dev
>
_______________________________________________
mylyn-docs-dev mailing list
mylyn-docs-dev@xxxxxxxxxxx
To change your delivery options, retrieve your password, or unsubscribe from this list, visit
https://dev.eclipse.org/mailman/listinfo/mylyn-docs-dev
--


David Green | VP of Architecture | Tasktop

email: david.green@xxxxxxxxxxx

--


David Green | VP of Architecture | Tasktop

email: david.green@xxxxxxxxxxx


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

Back to the top