ports/89300: [MAINTAINER] multimedia/bsdav: remove BROKEN

[Ion-Mihai Tetcu]

On Wed, 23 Nov 2005 00:02:55 +0100
Pav Lucistnik <pav at FreeBSD.org> wrote:

> Ion-Mihai Tetcu píše v st 23. 11. 2005 v 00:51 +0200:
> > On Tue, 22 Nov 2005 23:42:58 +0100
> > Pav Lucistnik <pav at FreeBSD.org> wrote:
> > 
> > > Ion-Mihai Tetcu píše v st 23. 11. 2005 v 00:34 +0200:
> > > > On Tue, 22 Nov 2005 22:20:01 GMT
> > > > Pav Lucistnik <pav at FreeBSD.org> wrote:
> > > > 
> > > > > Synopsis: [MAINTAINER] multimedia/bsdav: remove BROKEN
> > > > > 
> > > > > State-Changed-From-To: open->closed
> > > > > State-Changed-By: pav
> > > > > State-Changed-When: Tue Nov 22 22:19:30 GMT 2005
> > > > > State-Changed-Why: 
> > > > > Committed with fixes. Next time, what about running
> > > > > portlint(1)?
> > > > 
> > > > What was wrong ? ``^^'' = my comments
> > > > 
> > > > root at it> /usr/ports/multimedia/bsdav.my [0:32:16] 0
> > > >  # portlint -A
> > > 
> > > You should be running either `portlint -AN' for new ports or
> > > `portlint -C' for updates ...
> > 
> > Still, what was the problem ? I wanna learn, not make you loose
> > time, you know  :)
> > 
> > root at it> /usr/ports/multimedia/bsdav.my [0:46:11] 0
> >  # portlint -C
> > WARN: Makefile [26]: possible direct use of command "install"
> > found. use ${INSTALL_foobaa} instead. ^^ false
> > WARN: /usr/ports/multimedia/bsdav.my/files/patch-lib_bsdav_tables.c
> > [8]: includes possible RCS tag "$Id$". use binary mode (-ko) on
> > commit/import.
> > WARN: /usr/ports/multimedia/bsdav.my/files/patch-lib_bsdav_video_hw.c
> > [8]: includes possible RCS tag "$Id$". use binary mode (-ko) on
> > commit/import. ^^ are this two a problem ? WARN: no CVS
> > directories. Use -N to check a new port. ^^ 'casue I've copied the
> > port in bsdav.my; in the PR is OK (bsdav) 0 fatal errors and 4
> > warnings found.
> 
> I got different output. You're using latest and greatest portlint
> 2.8.0? I got the IGNORE capitalization message.

portlint-2.8.0_1

> Anyway, my changes:
> 
> First, you don't raise PORTREVISION on change that don't affect
> package.

Not compilable so no package so bump; yeh, my bad; it was compilable on
any machine with X installed (and I suspect the vast majority of users
of a TV-capture port are running X). Somehow things got twisted in my
brain, I'll be more careful next time. 

> Second, your IGNORE string does not make any sense when used. I fixed
> that.

OK, thanks.
 
> Third, you used OPTIONS incorrectly. I fixed that. For details, see
> chapter in PH.

I see that:
-.ifdef(WITH_X11)
+.if !defined(WITHOUT_X11)

OK, if one does not what X it will put make.conf WITHOUT_X11= yes; so
the port actually works (OPTIONS-wise) but one should use WITHOUT_X11
and this is said in PH.

1 tab -> 2 tabs change - style, my bad.


Thanks and sorry for putting you to work on my behalf,

-- 
IOnut - Unregistered ;) FreeBSD "user"
  "Intellectual Property" is   nowhere near as valuable   as "Intellect"

BOFH excuse #316:
Elves on strike (Why do they call EMAG Elf Magic?)