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

[Bug] jakarta dependency but referencing javax for imports #114

Open
CoryBarney opened this issue Apr 9, 2024 · 2 comments
Open

[Bug] jakarta dependency but referencing javax for imports #114

CoryBarney opened this issue Apr 9, 2024 · 2 comments
Assignees

Comments

@CoryBarney
Copy link

Been trying to update the version of postal on a project and noticed some dependency mismatches where the project is pulling in jakarta but importing javax.

Was wondering if it would be okay to update the imports to reference jakarta to future proof it for a bit?

@drewr drewr self-assigned this Apr 10, 2024
@drewr
Copy link
Owner

drewr commented Apr 10, 2024

Hey @CoryBarney! What deps conflicts are you getting?

@CoryBarney
Copy link
Author

Morning Drew @drewr, Sorry about the vague ticket ill get more details later when I swing back around to fighting with this but essentially without:

    <dependency>
      <groupId>com.draines</groupId>
      <artifactId>postal</artifactId>
      <version>2.0.5</version>
    </dependency>
    
    <dependency>
      <groupId>javax.mail</groupId>
      <artifactId>javax.mail-api</artifactId>
      <version>1.6.2</version>
  </dependency>

I end up getting a compilation error at line 1 message.clj stating javax.mail is not present as the dependency in the postal build pulls in the new none javax version of the library that I believe is due to licensing called jakarata:

project.clj

  :dependencies [[commons-codec "1.9"]
                 [com.sun.mail/jakarta.mail "1.6.5"]]

which has a class path of jakarta.* instead of what is referenced in the source code of javax.*

Ill try to get a PR out but it should just be a matter of updating the imports though I am not sure of the side effects personally

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

No branches or pull requests

2 participants