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

Add support for Eclipse-BundleShape in Manifest-Editor #864

Closed
HannesWell opened this issue Nov 2, 2023 · 4 comments · Fixed by #949
Closed

Add support for Eclipse-BundleShape in Manifest-Editor #864

HannesWell opened this issue Nov 2, 2023 · 4 comments · Fixed by #949
Labels
good first issue Good for newcomers

Comments

@HannesWell
Copy link
Member

The PDE Manifest-Editor should support the Eclipse-BundleShape header similiar to how it for example supports Bundle-ActivationPolicy.

In the Overview page there should be a check-box labeled with Unpack this plug-in during installation or similar.
And when it is checked the Eclipse-BundleShape; dir should be added. Unchecking it can simplify remove the header, since jar is the default.
The check-box should have a tool-tip indicating that this option is usually not required, only if the code relys on that embedded resources are available as files directly.

This is a follow-up on #730.

@HannesWell HannesWell added the good first issue Good for newcomers label Nov 2, 2023
@HannesWell HannesWell changed the title Add supprt for Eclipse-BundleShape in Manifest-Editor Add support for Eclipse-BundleShape in Manifest-Editor Nov 2, 2023
@laeubi
Copy link
Contributor

laeubi commented Nov 3, 2023

Instead of a checkbox it would be better to have a drop down (or Radiobuttons) with

  • default
  • jar
  • directory

see

for why one would explicitly configure the "default", also I'm not sure if it is really "Unpack this plug-in during installation" if one just installs it into OSGi (@tjwatson ?) I think this is only performed by P2.

@tjwatson
Copy link
Contributor

tjwatson commented Nov 3, 2023

for why one would explicitly configure the "default", also I'm not sure if it is really "Unpack this plug-in during installation" if one just installs it into OSGi (@tjwatson ?) I think this is only performed by P2.

Isn't the default value simply not specifying the header at all? The framework pays no attention to this header, so it must be p2 that does anything with it.

@laeubi
Copy link
Contributor

laeubi commented Nov 3, 2023

The framework pays no attention to this header, so it must be p2 that does anything with it.

That's what I mean, therefor I find "Unpack this plug-in during installation" a bit misleading, and maybe not suitable for the main Manifest tab at all...

@alshamams
Copy link
Contributor

Hi,
I intend to take up this issue.

alshamams added a commit to alshamams/eclipse.pde that referenced this issue Nov 29, 2023
Three modes are supported: jar, dir and default.
First two are self-explanatory, third option removes the header
which is the default behaviour.
Fixes: eclipse-pde#864
alshamams added a commit to alshamams/eclipse.pde that referenced this issue Dec 1, 2023
Three modes are supported: jar, dir and default.
First two are self-explanatory, third option removes the header
which is the default behaviour.
Fixes: eclipse-pde#864
alshamams added a commit to alshamams/eclipse.pde that referenced this issue Dec 5, 2023
Three modes are supported: jar, dir and default.
First two are self-explanatory, third option removes the header
which is the default behaviour.
Fixes: eclipse-pde#864
HannesWell pushed a commit to alshamams/eclipse.pde that referenced this issue Dec 5, 2023
Three modes are supported: jar, dir and default.
First two are self-explanatory, third option removes the header
which is the default behaviour.
Fixes: eclipse-pde#864
HannesWell pushed a commit to alshamams/eclipse.pde that referenced this issue Dec 6, 2023
Three modes are supported: jar, dir and default.
First two are self-explanatory, third option removes the header
which is the default behaviour.
Fixes: eclipse-pde#864
alshamams added a commit to alshamams/eclipse.pde that referenced this issue Dec 11, 2023
Three modes are supported: jar, dir and default.
First two are self-explanatory, third option removes the header
which is the default behaviour.
Co-authored-by: Hannes Wellmann <wellmann.hannes1@gmx.net>

Fixes: eclipse-pde#864
HannesWell pushed a commit to alshamams/eclipse.pde that referenced this issue Dec 22, 2023
Three modes are supported: jar, dir and default.
First two are self-explanatory, third option removes the header
which is the default behaviour.
Co-authored-by: Hannes Wellmann <wellmann.hannes1@gmx.net>

Fixes: eclipse-pde#864
laeubi pushed a commit to alshamams/eclipse.pde that referenced this issue Jan 27, 2024
Three modes are supported: jar, dir and default.
First two are self-explanatory, third option removes the header
which is the default behaviour.
Co-authored-by: Hannes Wellmann <wellmann.hannes1@gmx.net>

Fixes: eclipse-pde#864
laeubi pushed a commit to alshamams/eclipse.pde that referenced this issue Feb 11, 2024
Three modes are supported: jar, dir and default.
First two are self-explanatory, third option removes the header
which is the default behaviour.
Co-authored-by: Hannes Wellmann <wellmann.hannes1@gmx.net>

Fixes: eclipse-pde#864
HannesWell added a commit to alshamams/eclipse.pde that referenced this issue Feb 11, 2024
Three modes are supported: jar, dir and default.
First two are self-explanatory, third option removes the header
which is the default behavior.

Fixes: eclipse-pde#864

Co-authored-by: Hannes Wellmann <wellmann.hannes1@gmx.net>
HannesWell added a commit that referenced this issue Feb 11, 2024
Three modes are supported: jar, dir and default.
First two are self-explanatory, third option removes the header
which is the default behavior.

Fixes: #864

Co-authored-by: Hannes Wellmann <wellmann.hannes1@gmx.net>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
good first issue Good for newcomers
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants