From: David DEMELIER on 15 Jul 2010 16:50 2010/7/14 Sahil Tandon <sahil(a)freebsd.org>: > [mail/dspam maintainer Cc:'d] > > On Tue, 2010-07-13 at 09:46:04 +0200, David DEMELIER wrote: > >> 2010/7/13 Sahil Tandon <sahil(a)freebsd.org>: >> > On Mon, 2010-07-12 at 21:28:56 +0200, David DEMELIER wrote: >> > >> >> .if defined(WITH_MYSQL) >> >> WANT_MYSQL_VER= 55 ... >> >> .endif >> >> >> >> Then if the user wants a different version he still can change using : >> >> WITH_MYSQL_VER= 44. >> > >> > That change by itself will not work because in bsd.database.mk: >> > >> > Â .if defined(WITH_MYSQL_VER) && ${WITH_MYSQL_VER} != ${WANT_MYSQL_VER} >> > Â IGNORE= ... >> >> Then, explain me why do both php5 and php5-pgsql have not >> KNOBS/OPTIONS like dspam and work with this variable WITH_PGSQL_VER ? > > This is a good opportunity for you to learn about how the ports system > works. Â Compare the contents of both ports and also review > bsd.database.mk; if, after that, you still do not understand the current > behavior, let us know. > I'm writing a brand new mail/dspam port from scratch, if I finally finish it in the next fews days I will show you. The only problem I have for now is that there is no WITH_CYRUS_VER like variable so I can't make a per-user defined variable for which cyrus version to install. >> Because I meant, if someone made these KNOBS, that's would suppose >> they should be used. As shown just above it works with php5-pgsql. >> >> So if a variable could be used to specify a specific version, it must >> be used instead of a couple of option that set the package version. I >> think WITH_MYSQL_VER and WITH_PGSQL_VER were made for that. (There is >> one for apache too) > > I appreciate your frustration, but please realize that there are often > many ways to accomplish the same task. Â Trade-offs are involved. If you > believe a port can be improved, please (and now I'm repeating myself) > submit a patch! Â If you are unable to create a patch, then clearly > explain your idea in pseudo-code and submit it as a PR for the > maintainer's consideration. Â If you have trouble creating a patch, also > feel free to ping me off-list; I'm happy to work with you on this. > >> >> We must make a big KISS cleanup in the ports tree to make it as good >> >> as NetBSD' pkgsrc. >> > >> > Please send PRs or provide specific details; general comments like the >> > above are not helpful. >> >> Yes sometime I'm nasty with people, I'm sorry about that but when I >> saw this port I just didn't understand. > > What did you not understand? > >> I apologize for my english. > > No apology needed, as that is not the problem here. > > -- > Sahil Tandon <sahil(a)FreeBSD.org> > _______________________________________________ > freebsd-ports(a)freebsd.org mailing list > http://lists.freebsd.org/mailman/listinfo/freebsd-ports > To unsubscribe, send any mail to "freebsd-ports-unsubscribe(a)freebsd.org" > -- Demelier David _______________________________________________ freebsd-ports(a)freebsd.org mailing list http://lists.freebsd.org/mailman/listinfo/freebsd-ports To unsubscribe, send any mail to "freebsd-ports-unsubscribe(a)freebsd.org"
From: Doug Barton on 15 Jul 2010 18:52
On Thu, 15 Jul 2010, David DEMELIER wrote: > I'm writing a brand new mail/dspam port from scratch, if I finally > finish it in the next fews days I will show you. The only problem I > have for now is that there is no WITH_CYRUS_VER like variable so I > can't make a per-user defined variable for which cyrus version to > install. Sure you can, you just won't have it all handed to you on a silver platter. :) For example, look at how mail/alpine handles the selection of speller. hth, Doug -- Improve the effectiveness of your Internet presence with a domain name makeover! http://SupersetSolutions.com/ Computers are useless. They can only give you answers. -- Pablo Picasso _______________________________________________ freebsd-ports(a)freebsd.org mailing list http://lists.freebsd.org/mailman/listinfo/freebsd-ports To unsubscribe, send any mail to "freebsd-ports-unsubscribe(a)freebsd.org" |