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

From: Pavel Raiskup <praiskup(at)redhat(dot)com>
To: pgsql-jdbc(at)postgresql(dot)org
Cc: Vladimir Sitnikov <sitnikov(dot)vladimir(at)gmail(dot)com>, Pavel Kajaba <pkajaba(at)redhat(dot)com>, Stephen Nelson <stephen(at)eccostudio(dot)com>, Dave Cramer <pg(at)fastcrypt(dot)com>
Subject: Re: [pgjdbc] Implement JDBC specs via pre-processor step (#435)
Date: 2016-01-20 09:32:24
Message-ID: 3492059.LKk05KFoD2@nb.usersys.redhat.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-jdbc

Hi Vladimir, I'm kindly reconstructing the way-too-much trimmed conversation in
this thread:

Vladimir wrote:
> Pavel R. wrote:
> > Vladimir wrote:
> > > Pavel K. wrote:
> > > > Vladimir wrote:
> > > > > Pavel K. wrote:
> > > > > > Here [1] is my pull request
> > > > >
> > > > > How does it address 'fedora no internet' problem?
> > > >
> > > > All required maven dependencies are BuildDependencies in spec file.
> > > >
> > > > It means that everything is in fedora repositories in form of RPMS.
> > > >
> > > > So all maven dependencies are downloaded before build.
> > > >
> > > > This solves problem with missing waffle.
> > >
> > > Why do you need #495 change then?
> >
> > You are asking why we don't add waffle into Fedora? The Windows specific
> > 'jar' files can't be distributed in Fedora, most probably neither built
> > from source on/for Fedora. But please, substitute Fedora with any open
> > source GNU/Linux distribution,
>
> No. I am not.
>
> I am asking: "can you kindly provide a build log of a failed due to
> waffle build?"
> In other words, "a fedora build log before #495, and a fedora build
> log after #495 is merged in".
>
> Alternative: can you provide a test case that *fails* without #495?
>
> Unfortunately, current #495 has no tests behind. All tests pass
> without #495, thus the change does almost nothing.

Those questions sound much more concrete ;). Pavel, can we provide failed
build logs or other info? I've posted issues with maven in this thread [1],
I bet with waffle it is similar situation -- from quick look at Pavel's --
it looks he tries to make the waffle "optional" dependancy -- but I
haven't done review.

[1] http://www.postgresql.org/message-id/4027547.QFGZ3nUSVM@nb.usersys.redhat.com

Pavel

In response to

Browse pgsql-jdbc by date

  From Date Subject
Next Message Vladimir Sitnikov 2016-01-20 09:42:13 Re: [pgjdbc] Implement JDBC specs via pre-processor step (#435)
Previous Message Pavel Kajaba 2016-01-20 09:24:24 Re: [pgjdbc] Implement JDBC specs via pre-processor step (#435)