On Mon, Jan 9, 2012 at 4:59 PM, Alan DeKok <aland@deployingradius.com> wrote:
Fajar A. Nugraha wrote:
I'm trying to build master branch (last commit d1dfb80) on Ubuntu natty (gcc-4.5.2) with a simple ./configure && make, and the build failed with this error:
I've pushed a tiny fix.
Thanks. I kinda suspect it was something like that :D Some more observations (just curious, not important) - rlm_protocol_filter is now removed. No problem though, it's explained in git log - radeapclient is not built by default. There's a comment about it on git log, but no additional explanation. Is there a documentation on the reasoning somewhere? What IS important, is that I noticed sql.conf and eap.conf is now moved to modules/. AFAIK all files in modules/ is included and parsed by the server. This might be a problem in the following scenarios: - user install FR3 package (e.g. rpm/deb) - user modifies modules/sql or modules/eap - user upgrades FR3 package, and since the two files changed, new files will be created (e.g. modules/sql.rpmnew or modules/sql.dpkg-new) - both files (e.g. modules/sql and modules/sql.rpmnew) is included and parsed by the server, and both define the same module instance. This might mean the wrong instance is used, or at minimum create a confusion. What is the recommended way to solve this problem? Should packagers warn users "don't edit any file in modules/*, instead copy to a new file and assign new instance name"? -- Fajar