Re: [PATCH 4/4] BSE: install additional #include files required to build SpectMorph

[Tim Janik <timj gnu org>]

Hm,

an "internal" header like bse-internals.hh should obviously not be installed.
Looking at it, it's mostly including otherwise public math headers though.
So I winder why it got named like this in the first place. I guess the header
should be renamed when installed, care to dig up its uses/history and
suggest a better name?


On 18.10.2015 16:42, Stefan Westerfeld wrote:
> Signed-off-by: Stefan Westerfeld <stefan space twc de>
> ---
>  bse/Makefile.am | 3 +--
>  1 file changed, 1 insertion(+), 2 deletions(-)
>
> diff --git a/bse/Makefile.am b/bse/Makefile.am
> index 3668938..76c2a09 100644
> --- a/bse/Makefile.am
> +++ b/bse/Makefile.am
> @@ -33,7 +33,7 @@ bse_public_headers = $(strip \
>       bsebus.hh               bsecategories.hh        \
>       bsefilter.hh                    bsebiquadfilter.hh      \
>       bseconstant.hh          bseconstvalues.hh       bsecontainer.hh                 bsecontextmerger.hh \
> -     bseclientapi.hh         \
> +     bseclientapi.hh         bseserverapi.hh         bse-internals.hh                \
>                               bseeditablesample.hh    bseenums.hh                     bsegconfig.hh \
>       bseglobals.hh           bseglue.hh              bseitem.hh                      bsejanitor.hh \
>       bsemain.hh              bsemath.hh              bsemathsignal.hh                        bseladspa.hh \
> @@ -129,7 +129,6 @@ EXTRA_DIST += $(strip \
>       bsewave.header                                          \
>       gsl-fftgen.pl   gsl-fftconf.sh  gsloscillator-aux.cc    \
>       gslincluder.hh  gslwaveosc-aux.cc                       \
> -     bse-internals.hh                                        \
>  )
>  
>  #


-- 
Yours sincerely,
Tim Janik

https://testbit.eu/timj/
Free software author and speaker.