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

Added composite repo and removed the sdks #498

Merged
merged 3 commits into from
Dec 9, 2024

Conversation

keinhaar
Copy link
Contributor

This is intended for Release 4.2.
It add's composite repo files and removes the sdk's, which can be build separately by the gwt-sdk-plugins project.
The Update Site should look like this:

UPDATE_URL/
    compositeContent.xml
    compositeArtifacts.xml
    gwt-eclipse-plugin/
        content.xml
        artifacts.xml
        plugins/
        features/
        ...
    gwt-sdk-plugins/
        content.xml
        artifacts.xml
        plugins/
        features/
        ...

I think, we should also have a latest folder for the latest version, because otherwise we always have to update documentation pages and the marketplace entry.
Another thing to think about: Should we move the Update Site to our own documentation page? Otherwise we depend on the GWT Project itself to update the repo.

protoism
protoism previously approved these changes Nov 27, 2024
Copy link
Contributor

@protoism protoism left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm simply looking forward to seeing this in production.
Well done.

Copy link
Contributor

@protoism protoism left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm a bit doubtful about where you plan to host plugins and sdk

type='org.eclipse.equinox.internal.p2.artifact.repository.CompositeArtifactRepository'
version='1.0.0'>
<properties size='1'>
<property name='p2.timestamp' value='1665755219001'/>
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

What is this timestamp? What's the purpose of it?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The timestamp is just something eclipse created. I don't know what's it good for. Plugins are installable even if the timestamp is always the same.

<property name='p2.timestamp' value='1665755219001'/>
</properties>
<children size='2'>
<child location='https://gwt-plugins.github.io/documentation/repo/gwt-eclipse-plugin'/>
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm not clear if we're talking about documentation or plugins.
I'm afraid you intend to move the plugin storage to github pages which... looks like a dangerous choice.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I changed the location to plugins.gwtproject.org/eclipse in the second commit as discussed with colin.
Nothing will be on the github.io page except the documentation itself.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Go ahead, then!

@keinhaar
Copy link
Contributor Author

keinhaar commented Dec 1, 2024

Sorry, i needed to force push, because git did not stop telling me it cannot push because i'm behind the origin, which was not true (Still after rebasing, fetching, pulling multiple times)
The only changes are in the second commit, where i updated the URLs in the composite Site files.

protoism
protoism previously approved these changes Dec 2, 2024
@keinhaar
Copy link
Contributor Author

keinhaar commented Dec 9, 2024

@protoism Could you please verify again. I needed to merge the index.html of the update site. nothing else changed.

@keinhaar keinhaar merged commit ccb8f3e into gwt-plugins:main Dec 9, 2024
1 check passed
@keinhaar keinhaar deleted the splitOffSDKs branch December 9, 2024 16:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants