Re: [Buildroot] [PATCH v9] package/freeradius-server: new package

[Arnout Vandecappelle <arnout@mind.be>]

On 04/01/2022 22:55, Matthew Weber via buildroot wrote:
> From: David GOUARIN <dgouarin@gmail.com>
> 
> FreeRADIUS is an open source server which implements
> a protocol for remote user Authorization, Authentication
> and Accounting.
> 
> Signed-off-by: David GOUARIN <dgouarin@gmail.com>
> Signed-off-by: Kalpesh Panchal <kalpesh.panchal@rockwellcollins.com>
> Signed-off-by: Matt Weber <matthew.weber@collins.com>

  Finally applied to master, thanks. I still made a large number of changes.

      - remove second patch, superseded by other patches;
      - add upstream links to patches;
      - add 3 more patches to avoid looking in host directories;
      - explicitly add dependency on !static inherited from talloc (redundant
        with glibc, but future-safe);
      - simplify Config.in comment;
      - check hash with PGP signature;
      - add conf opts for runtime paths;
      - add conf opts to disable unsupported modules;
      - add more optional dependencies;
      - enable/disable all modules that use a dependency;
      - search defaults file in /etc/default, not /etc/sysconfig.

  I also created an upstream pull request for all the patches that are not yet 
upstream.

  Regards,
  Arnout

> ---
> Changes
> v8 -> v9:
>   - A systemd unit is installed if systemd is enabled. The default provided
>     by `freeradius-server/redhat/radiusd.service` was not used because of
>     extra options not compatible with Buildroot. The unit is instead based
>     on a simplified version of that upstream file.
> 
> v7 -> v8:
>    - added dependencies to include libpcap
>    - version bump to 3.0.25
> 
> v6 -> v7
>    - resolve rlm_sql module compilation error (Newer GCC)
>      (remove mysql_version.h as mysql.h is included)
> 
> v5 -> v6:
>    - cleaned up patches to all be git formatted
>    - added depends on BR2_USE_MMU
>    - rebased on master
>    - added CPE info
> 
> v4 -> v5:
>    - better package patches formatting
> 
> v3 -> v4:
>    Taken in account Thomas review
>    - fix build with BR2_COMPILER_PARANOID_UNSAFE_PATH
>    - import patches inside package directory instead of downloading them
>    - remove config.cache file, pass the answers on the command line instead
>    - add comments for weird TARGET_DIR and R directives
>    - remove broken python optional feature. Managed to get it to build
>      with python 2.7, not 3.0 and it still fails basic testing
> 
> v2 -> v3:
>    - As suggested by Matt, merge with pending upstream patch. Included most of it,
>      including patches based on Yocto work :
>      http://cgit.openembedded.org/meta-openembedded/tree/meta-networking/recipes-connectivity/freeradius/files
>      Didn't kept all the --without directive, as they do not seem useful, nor
>      the custom INSTALL_TARGET_CMDS.
> 
> v1 -> v2:
>     - fix invalid characters in Config.in
[snip]
_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot