Re: [pgjdbc] Implement JDBC specs via pre-processor step (#435)

From: Jeremy Whiting <jwhiting(at)redhat(dot)com>
To: Vladimir Sitnikov <sitnikov(dot)vladimir(at)gmail(dot)com>, Álvaro Hernández Tortosa <aht(at)8kdata(dot)com>
Cc: Pavel Kajaba <pkajaba(at)redhat(dot)com>, Stephen Nelson <stephen(at)eccostudio(dot)com>, List <pgsql-jdbc(at)postgresql(dot)org>, Pavel Raiskup <praiskup(at)redhat(dot)com>, Dave Cramer <pg(at)fastcrypt(dot)com>
Subject: Re: [pgjdbc] Implement JDBC specs via pre-processor step (#435)
Date: 2016-01-22 12:30:44
Message-ID: 56A220F4.2050904@redhat.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-jdbc

On 22/01/16 07:42, Vladimir Sitnikov wrote:
> Álvaro>Run "mvn dependency:list"
>
> mvn dependency:analyze-report allows to capture "Used but undeclared
> dependencies" and "Unused but declared dependencies" as well.
>
> Álvaro>- xml-resolver: pretty old code
>
> I've no idea what it is doing there. It is marked as compile-time
> dependency, however the code compiles without it just fine.
> I think it can be just dropped.
xml-resolver was added [1] as a dependency to support using an XML
catalogue with docbook.
Since docbook has been removed [2] the xml-resolver dependency is safe
to remove [3] also.

Jeremy

[1] https://github.com/pgjdbc/pgjdbc/pull/284
[2]
https://github.com/pgjdbc/pgjdbc/commit/92b65f04de1240b24bd4f71e34903422656660c5
[3] https://github.com/pgjdbc/pgjdbc-parent-poms/pull/3
> Álvaro> OK, so here's how to reproducible check the dependencies of
> the maven project:
>
> What we miss is:
> 1) Fedora-like CI environment. It makes little sense optimizing
> pgjdbc's dependencies if those dependencies are already present in
> Fedora-like build boxes.
> Is there a CI for Fedora builds?
> How do we validate that "the specific changes to pgjdbc" indeed help
> Fedora packaging?
>
> 2) Requirements. I'm sure the first dozen of builds will fail due to
> packaging limitations. It is much better to know the limitations
> beforehand than running into them at Fedora CI builds.
>
> Vladimir
>
>

In response to

Browse pgsql-jdbc by date

  From Date Subject
Next Message Dave Cramer 2016-01-22 12:32:50 Re: Wrong link not pointing to the release tarball
Previous Message Pavel Raiskup 2016-01-22 12:30:14 Wrong link not pointing to the release tarball