Bug 1217395 - update maven notation
Summary: update maven notation
Keywords:
Status: CLOSED RAWHIDE
Alias: None
Product: Fedora
Classification: Fedora
Component: postgresql-jdbc
Version: 23
Hardware: Unspecified
OS: Unspecified
unspecified
unspecified
Target Milestone: ---
Assignee: Pavel Raiskup
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks: 1328868 1358134
TreeView+ depends on / blocked
 
Reported: 2015-04-30 09:46 UTC by gil cattaneo
Modified: 2016-07-21 13:01 UTC (History)
2 users (show)

Fixed In Version:
Clone Of:
Environment:
Last Closed: 2016-07-21 12:29:05 UTC
Type: Bug
Embargoed:


Attachments (Terms of Use)
update maven notation (5.98 KB, patch)
2015-04-30 09:48 UTC, gil cattaneo
no flags Details | Diff
update maven notation (6.23 KB, patch)
2016-06-19 15:22 UTC, gil cattaneo
no flags Details | Diff
fix postgresql-jdbc.jar symlink using javapackages macros (9.09 KB, patch)
2016-07-21 06:50 UTC, gil cattaneo
no flags Details | Diff

Description gil cattaneo 2015-04-30 09:46:27 UTC
this and newer release should be with this maven notation 
org.postgresql:postgresql:9.4-1200-jdbc41

Comment 1 gil cattaneo 2015-04-30 09:48:09 UTC
Created attachment 1020511 [details]
update maven notation

- generate pom file
- use new javapackages macros
- introduce %%license macro
- remove some rpmlint problems (mixed-use-of-spaces-and-tabs)
- adapt to current guideline
- add alias alias for backward compatibility

Task info: http://koji.fedoraproject.org/koji/taskinfo?taskID=9606006

Comment 2 Jan Kurik 2015-07-15 14:13:13 UTC
This bug appears to have been reported against 'rawhide' during the Fedora 23 development cycle.
Changing version to '23'.

(As we did not run this process for some time, it could affect also pre-Fedora 23 development
cycle bugs. We are very sorry. It will help us with cleanup during Fedora 23 End Of Life. Thank you.)

More information and reason for this action is here:
https://fedoraproject.org/wiki/BugZappers/HouseKeeping/Fedora23

Comment 3 Pavel Raiskup 2016-01-25 12:35:43 UTC
Thanks for the report.

This all should be fixed by rebase to new version of postgresql-jdbc.  Once
done, I'll close this as duplicate of bug 1196445.

Comment 4 gil cattaneo 2016-01-25 13:52:17 UTC
(In reply to Pavel Raiskup from comment #3)
> Thanks for the report.
> 
> This all should be fixed by rebase to new version of postgresql-jdbc.  Once
> done, I'll close this as duplicate of bug 1196445.

It's ok for me. Thanks

Comment 5 gil cattaneo 2016-06-19 15:22:45 UTC
Created attachment 1169531 [details]
update maven notation

- fix rhbz#1217395
- generate pom file
- use new javapackages macros
- introduce %license macro
- disable doclint
- remove some rpmlint problems (mixed-use-of-spaces-and-tabs)
- adapt to current guideline

Task info: http://koji.fedoraproject.org/koji/taskinfo?taskID=14565015

Comment 6 Pavel Raiskup 2016-07-20 08:23:07 UTC
gil, sorry for the delay -- could you please rebase your patches against
actual 'master' branch?

Comment 7 gil cattaneo 2016-07-20 08:46:30 UTC
(In reply to Pavel Raiskup from comment #6)
> gil, sorry for the delay -- could you please rebase your patches against
> actual 'master' branch?

No, problems. but is not possible merge the changes in https://bugzilla.redhat.com/show_bug.cgi?id=1358134#c2 in other branches?

Comment 8 Pavel Raiskup 2016-07-20 09:04:06 UTC
I would like to avoid merging this, what exactly you would like to have
in stable fedoras?

Comment 9 Pavel Raiskup 2016-07-20 09:05:09 UTC
Also, I've removed the postgresql-jdbc.jar symlink, which will probably
break some 'weka' runtime scripts.  Should I restore that symlink?

There is similar issue with libreoffice, I'll try to contact maintainers.

Comment 10 Pavel Raiskup 2016-07-20 11:42:12 UTC
I've reverted the postgresql-jdbc.jar symlink.

Comment 11 gil cattaneo 2016-07-20 17:57:35 UTC
(In reply to Pavel Raiskup from comment #10)
> I've reverted the postgresql-jdbc.jar symlink.

try with
%mvn_file org.postgresql:postgresql %{name}/postgresql %{name}
before %mvn_package macro

Comment 12 gil cattaneo 2016-07-20 17:58:53 UTC
(In reply to gil cattaneo from comment #11)
> (In reply to Pavel Raiskup from comment #10)
> > I've reverted the postgresql-jdbc.jar symlink.
> 
> try with
# compat symlink: requested by dtardon (libreoffice), reverts part of
# 0af97ce32de877 commit.
> %mvn_file org.postgresql:postgresql %{name}/postgresql %{name}
> before %mvn_package macro

and remove

pushd $RPM_BUILD_ROOT%{_javadir}
# compat symlink: requested by dtardon (libreoffice), reverts part of
# 0af97ce32de877 commit.
ln -s %{name}/postgresql.jar postgresql-jdbc.jar
popd

Comment 13 Pavel Raiskup 2016-07-21 04:48:36 UTC
gil, can you please re-attach patch with requested changes?  That would really
simplify things.

But just please keep the comment saying why the compat postgresql-jdbc.jar
exists.

Comment 14 gil cattaneo 2016-07-21 06:50:00 UTC
Created attachment 1182329 [details]
fix postgresql-jdbc.jar symlink using javapackages macros

- remove some rpmlint problems (mixed-use-of-spaces-and-tabs)
- adapt to current guideline
- install doc and license file in parent-poms sub package
- simplified runselftest check

Task info: http://koji.fedoraproject.org/koji/taskinfo?taskID=14965976

Comment 15 Pavel Raiskup 2016-07-21 12:29:05 UTC
Thanks!  Gil, pushed with one slight modification -- I removed the
tab2space change, as the tabs are in majority of PostgreSQL related packages
and I don't want to change it with respect to history of that package :)
(it is kind of sentimental).  Also, I'm not sure -- my F24 version of rpmlint
was silent before and so it is now (the mixed-use-of-spaces-and-tabs).  Is
that still an issue?

I'm closing and thanks again!  If there are some issues, feel free to discuss.
http://pkgs.fedoraproject.org/cgit/rpms/postgresql-jdbc.git/commit/?id=668b4398f9c43

Comment 16 gil cattaneo 2016-07-21 13:01:20 UTC
(In reply to Pavel Raiskup from comment #15)
> Thanks!  Gil, pushed with one slight modification -- I removed the
> tab2space change, as the tabs are in majority of PostgreSQL related packages
> and I don't want to change it with respect to history of that package :)
> (it is kind of sentimental). 
my patch was only some suggestion for adapt spec file with the current guideline
> was silent before and so it is now (the mixed-use-of-spaces-and-tabs).  Is
> that still an issue?
Is not an issues but it makes the file spec a little more clean and to eliminate some problems, in less, with rpmlint
> I'm closing and thanks again!  If there are some issues, feel free to
> discuss.
> http://pkgs.fedoraproject.org/cgit/rpms/postgresql-jdbc.git/commit/
> ?id=668b4398f9c43
again thanks
regards


Note You need to log in before you can comment on or make changes to this bug.