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

Consider publishing an updated release #30

Open
fwgreen opened this issue Nov 29, 2018 · 9 comments
Open

Consider publishing an updated release #30

fwgreen opened this issue Nov 29, 2018 · 9 comments

Comments

@fwgreen
Copy link

fwgreen commented Nov 29, 2018

I've been trying to build the current master, but I'm encountering problems at runtime:

Error occurred during initialization of boot layer
java.lang.module.FindException: Module client not found, required by org.postgresql.adba

I'm not very good with Maven so I'm giving up.

@alexanderkjall
Copy link
Collaborator

Hi

This is something that needs to be handled before the next release, the problem is that the scram project doesn't have module support and I didn't want to reimplement all that crypto code.

I have opened a pull request in that project, but so far no response: ongres/scram#5

I need to either fork and do my own release of the scram project or copy those source files into this project and both of those options come with drawbacks.

@fwgreen
Copy link
Author

fwgreen commented Nov 29, 2018

Maybe it would be enough to add an Automatic-Module-Name to the manifest of each project?

<plugin>
  <groupId>org.apache.maven.plugins</groupId>
  <artifactId>maven-jar-plugin</artifactId>
  <configuration>
    <archive>
      <manifestEntries>
        <Automatic-Module-Name>com.ongres.scram.client</Automatic-Module-Name>
      </manifestEntries>
    </archive>
  </configuration>
</plugin>

The maintainers might be more amenable to this, less drastic, change.

@alexanderkjall
Copy link
Collaborator

That is a great suggestion, I'll update my pull request with that as soon as I get some free time.

On another note, what version of maven do you use to build? I have no problem building master with maven 3.6.0 and java 10.

@fwgreen
Copy link
Author

fwgreen commented Nov 29, 2018

I'm also using Maven 3.6.0, but on Java 11. I think support for 10 ended when 11 was released.

@fwgreen
Copy link
Author

fwgreen commented Feb 24, 2019

@alexanderkjall Has there been any recent communication between the pgjdbc team and the ongres scram maintainers? I get the feeling that https://github.com/ongres/scram has been abandoned.

@davecramer
Copy link
Member

@fwgreen What is needed in scram ?

@fwgreen
Copy link
Author

fwgreen commented Feb 25, 2019

@davecramer It seems to not be compatible with Java 11 and the JPMS in its current form.

@davecramer
Copy link
Member

I'm pinging Alvaro.. will let you know

@alexanderkjall
Copy link
Collaborator

I emailed Alvaro before christmas and asked if he was willing to merge the request but was busy with real life and would get around to it.

Maybe he forgot it, otherwise I guess that I will need to fork that project in some way.

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

3 participants