Bug 127147 - Update port: textproc/lucene
Summary: Update port: textproc/lucene
Status: Closed FIXED
Alias: None
Product: Ports & Packages
Classification: Unclassified
Component: Individual Port(s) (show other bugs)
Version: Latest
Hardware: Any Any
: Normal Affects Only Me
Assignee: Martin Wilke
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2008-09-06 14:20 UTC by Gerrit Beine
Modified: 2008-10-31 13:00 UTC (History)
0 users

See Also:


Attachments
file.diff (1.50 KB, patch)
2008-09-06 14:20 UTC, Gerrit Beine
no flags Details | Diff
lucene.contrib.diff (1.34 KB, patch)
2008-10-31 12:36 UTC, mi+k
no flags Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Gerrit Beine 2008-09-06 14:20:03 UTC
Added option to install contrib JARs with the port.

Thanks to Mikhail T.
Comment 1 Edwin Groothuis freebsd_committer freebsd_triage 2008-09-06 14:20:12 UTC
Responsible Changed
From-To: freebsd-ports-bugs->miwi

miwi@ wants his PRs (via the GNATS Auto Assign Tool)
Comment 2 dfilter service freebsd_committer freebsd_triage 2008-09-06 22:11:10 UTC
miwi        2008-09-06 21:11:02 UTC

  FreeBSD ports repository

  Modified files:
    textproc/lucene      Makefile 
  Log:
  - Added option to install contrib JARs with the port.
  
  PR:             127147
  Submitted by:   Gerrit Beine <gerrit.beine@gmx.de> (maintainer)
  
  Revision  Changes    Path
  1.4       +32 -2     ports/textproc/lucene/Makefile
_______________________________________________
cvs-all@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/cvs-all
To unsubscribe, send any mail to "cvs-all-unsubscribe@freebsd.org"
Comment 3 Martin Wilke freebsd_committer freebsd_triage 2008-09-06 22:11:15 UTC
State Changed
From-To: open->closed

Committed. Thanks!
Comment 4 mi 2008-09-07 22:34:18 UTC
> > Hi all,
> > good patch, please approve, Gerrit!
> >
> > -- Nakata Maho http://accc.riken.jp/maho/
 
> I did some additions to the patch and submitted it as PR.

Gerrit, as submitted, the contributed-JARs are not installed
by default. This keeps the port useless for the likes of OOo,
which rely on some of these JARs.

Can I flip the default to `on'? Thanks!

	-mi
Comment 5 mi+k 2008-10-31 12:36:43 UTC
Hi, Gerrit!

You've never replied to this e-mail of mine, and the install of "contrib" JARs 
remains OFF by default. The PR 127147 was committed without testing and 
without attribution (which is good, because the WITH_CONTRIB-case was 
broken), AND the port has since been upgraded again -- to 2.4.0 -- once again 
missing an opportunity to flip the default without messing with PORTREVISION.

Can I commit the attached diff to:

	1. Make the WITH_CONTRIB-case work.
	2. Make WITH_CONTRIB the default.

please? Thanks. Yours,

	-mi

I wrote:
= > > Hi all,
= > > good patch, please approve, Gerrit!
= > >
= > > -- Nakata Maho http://accc.riken.jp/maho/
=  
= > I did some additions to the patch and submitted it as PR.
= 
= Gerrit, as submitted, the contributed-JARs are not installed
= by default. This keeps the port useless for the likes of OOo,
= which rely on some of these JARs.
= 
= Can I flip the default to `on'? Thanks!
= 
= 	-mi
Comment 6 Gerrit Beine 2008-10-31 12:58:25 UTC
Am Freitag, 31. Oktober 2008 13:36:43 schrieb Mikhail Teterin:
> Hi, Gerrit!
>
> You've never replied to this e-mail of mine, and the install of "contrib"
> JARs remains OFF by default. The PR 127147 was committed without testing
> and without attribution (which is good, because the WITH_CONTRIB-case was
> broken), AND the port has since been upgraded again -- to 2.4.0 -- once
> again missing an opportunity to flip the default without messing with
> PORTREVISION.
>
> Can I commit the attached diff to:
>
> 	1. Make the WITH_CONTRIB-case work.
> 	2. Make WITH_CONTRIB the default.
>
> please? Thanks. Yours,

Hi,

if it helps you, switch it on.

So long...

Gerrit