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

Remove the JBoss reference from copyright and license headers #625

Closed
waynebeaton opened this issue Jun 7, 2022 · 10 comments · Fixed by #654 or #655
Closed

Remove the JBoss reference from copyright and license headers #625

waynebeaton opened this issue Jun 7, 2022 · 10 comments · Fixed by #654 or #655
Milestone

Comments

@waynebeaton
Copy link
Contributor

Many of the source files include this text at the very top of the file, as part of the copyright and license header:

JBoss, Home of Professional Open Source

Please remove it.

Some time shortly after the next release would be good timing.

@starksm64
Copy link
Contributor

OK, but could you explain why this is being requested? That header file has been in place since the project inception.

@waynebeaton
Copy link
Contributor Author

  1. Jakarta CDI is an Eclipse/Jakarta EE project, not a JBoss Project.
  2. The statement is effectively an advertisement for a specific vendor.
  3. It is suggestive of a particular implementation of the specification being more special than other implementations.
  4. There's no technical or legal reason for it to be there.

If you're wondering "why now?"... I only just noticed it. It never occurred to us before now to look for this sort of thing when we do IP checks.

@manovotn
Copy link
Contributor

I've linked this to the list of future version issues as it seems nobody is keen on sacrificing their time to it now.

@waynebeaton
Copy link
Contributor Author

$ find . -type f -print0 | xargs -0 sed -E -i "/^ \* JBoss, Home of Professional Open Source$/d"
$ find . -type f -print0 | xargs -0 sed -E -i "/^~ JBoss, Home of Professional Open Source$/d"
$_

There are two files that appear to have managed headers. My best guess is that some automated formatting was applied. We could sed these, but I'm thinking that some manual reformatting is probably preferred.

  • ./api/src/main/resources/beans_1_0.xsd
  • ./api/src/main/resources/beans_1_1.xsd

Shall I submit a pull request?

@manovotn
Copy link
Contributor

Shall I submit a pull request?

By all means, a PR is always welcome :)

@manovotn
Copy link
Contributor

Shall I submit a pull request?

@waynebeaton a friendly reminder in case you find a chunk of time to send that PR :)

@Ladicek
Copy link
Contributor

Ladicek commented Feb 28, 2023

We should probably have a linter for license headers.

@waynebeaton
Copy link
Contributor Author

Pull request has been delivered.

I ran the build and the tests all pass.

I hesitated to change the XSD files, but mostly because I don't really understand how they're disseminated and the impact of a non-functional change. Is there any reason why we shouldn't change these files?

@manovotn manovotn linked a pull request Mar 1, 2023 that will close this issue
@manovotn
Copy link
Contributor

manovotn commented Mar 1, 2023

I hesitated to change the XSD files, but mostly because I don't really understand how they're disseminated and the impact of a non-functional change. Is there any reason why we shouldn't change these files?

Do you mean beans.xsd files? I only see this vendor line in the two oldest ones BTW. And I don't know why changing them would be an issue.

@waynebeaton
Copy link
Contributor Author

Okay, I've created a new pull request #655 to address the remaining references in headers.

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 a pull request may close this issue.

4 participants