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 the default 'Implementation-*' headers to the manifest #879

Merged
merged 1 commit into from
Apr 21, 2014
Merged

Add the default 'Implementation-*' headers to the manifest #879

merged 1 commit into from
Apr 21, 2014

Conversation

fhuberts
Copy link
Contributor

Added headers are:
Implementation-Title: ${project.name}
Implementation-Version: ${project.version}
Implementation-Vendor-Id: ${project.groupId}
Implementation-Vendor: ${project.organization.name}

Related to #878

Signed-off-by: Ferry Huberts [email protected]

Added headers are:
Implementation-Title:     ${project.name}
Implementation-Version:   ${project.version}
Implementation-Vendor-Id: ${project.groupId}
Implementation-Vendor:    ${project.organization.name}

Related to #878

Signed-off-by: Ferry Huberts <[email protected]>
@kcooney
Copy link
Member

kcooney commented Apr 19, 2014

Wow, that was fast and simple!

Could you build the jar locally, print out the manifest, and add it as a comment this pull?

@fhuberts
Copy link
Contributor Author

no problem:

Manifest-Version: 1.0
Implementation-Vendor: JUnit
Implementation-Title: JUnit
Implementation-Version: 4.12-SNAPSHOT
Implementation-Vendor-Id: junit
Built-By: ferry
Build-Jdk: 1.7.0_55
Created-By: Apache Maven 3.1.1
Archiver-Version: Plexus Archiver

@kcooney
Copy link
Member

kcooney commented Apr 20, 2014

looks good to me

I believe @stefanbirkner and @marcphilipp are more knowledgeable about Maven, so I'll have one of them merge this.

@marcphilipp
Copy link
Member

LGTM. @stefanbirkner: Any objections?

@stefanbirkner
Copy link
Contributor

LGTM

stefanbirkner added a commit that referenced this pull request Apr 21, 2014
Add the default 'Implementation-*' headers to the manifest
@stefanbirkner stefanbirkner merged commit 62dd509 into junit-team:master Apr 21, 2014
@fhuberts fhuberts deleted the implementationheaders branch April 21, 2014 19:59
@fhuberts
Copy link
Contributor Author

thanks guys!

@stefanbirkner stefanbirkner added this to the 4.12 milestone Apr 21, 2014
@stefanbirkner
Copy link
Contributor

@fhuberts Could you please make a note at https://github.com/junit-team/junit/wiki/4.12-release-notes

@fhuberts
Copy link
Contributor Author

done

@stefanbirkner
Copy link
Contributor

Thanks.

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.

None yet

4 participants