Skip to main content
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index] [List Home]
Re: [jakartaee-tck-dev] Preparing to hack with ReWrite...


On 11/20/22 8:31 PM, Olivier Lamy wrote:
I have started some work. 
But is there any reason to split runtime and glassfishtck?

On a case by case basis, IMO we should move sources that we don't think are going to be needed by the actual refactored TCK tests, into a `to_be_deleted_later` folder.

It would also be good to isolate the GlassFish TCK into a separate folder that other test sources do not reference. 

Trying to have a clean (e.g at least compiling) I have to re enable those modules.
And there are some circular dependencies between both.
At the end of the day, those modules will be removed?

I think the test harness and other runtime sources will be removed as they serve no purpose. 

I'm not sure about the GlassFish TCK porting kit.  David, Guru, Alwin do you have a preference for where the GlassFish TCK porting kit should live? 

Scott



On Sun, Nov 20, 2022 at 4:53 PM Olivier Lamy <olamy@xxxxxxxxxxx> wrote:
Hi,
I'm not sure adding more tooling will fix anything (except making it more complicated by adding more tooling :)).
I have some issues on how the current tckrefactor build structure (e.g pom content), I have some changes in my branch with the servlet tck using arquilian. [1]
But I didn't to fix too many things in this branch as the merge/rebase will be an even worst nightmare for me :) 
Some content I have in mind which looks wrong (or I don't understand the reason) to my (long) Maven experience.
Why having everything in <dependencies> [2] in parent pom, this mean by example servlet or jsp will have batch or jms as dependency?
Why having this section with m-enforcer-p [3] is every poms?
I don't understand the need of using of build-helper-m-p [4] in every poms? we probably only need to have the sources in the standard Maven location.

If your concern is about having a clean build (e.g no compile issue) in the tckrefactor branch if you need a volunteer I can have a try for the next few days.

cheers
Olivier 



On Sat, Nov 19, 2022 at 12:54 AM Scott Marlow <smarlow@xxxxxxxxxx> wrote:

Hi,

I am finding that in order to use ReWrite [1] to make source changes in our `tckrefactor` branch, we need to first get to zero build failures.  My `rewrite` topic branch [2] has some changes but more changes are needed to resolve build failures such as [3].  My question for all of you is how can we best get the `tckrefactor` branch building cleanly? 

Is there anything that I can do to help others to participate in the clean up? 

Scott

[1] https://docs.openrewrite.org/getting-started
[2] https://github.com/scottmarlow/jakartaee-tck/tree/rewrite
[3] https://gist.github.com/scottmarlow/a2b388e0be13bb51c4b33f263ccd7ce7
[4] https://github.com/jakartaee/platform-tck/tree/tckrefactor

_______________________________________________
jakartaee-tck-dev mailing list
jakartaee-tck-dev@xxxxxxxxxxx
To unsubscribe from this list, visit https://www.eclipse.org/mailman/listinfo/jakartaee-tck-dev


--
Olivier


--
Olivier

_______________________________________________
jakartaee-tck-dev mailing list
jakartaee-tck-dev@xxxxxxxxxxx
To unsubscribe from this list, visit https://www.eclipse.org/mailman/listinfo/jakartaee-tck-dev

Back to the top