Re: [PATCH fbtest] pnmtohex: Optionally use pkg-config for netpbm

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Tue, Mar 28, 2023 at 2:20 PM Helge Deller <deller@xxxxxx> wrote:
> * Geert Uytterhoeven <geert@xxxxxxxxxxxxxx>:
> > As of libnetpbm11 in Debian/Ubuntu, the netpbm header files are no
> > longer located in the root include directory, but in a netbpm
> > subdirectory.  Fortunately the same version added support for
> > pkg-config.
> >
> > Support both old and new systems by using pkg-config, when available.
> >
> > Reported-by: Helge Deller <deller@xxxxxx>
> > Signed-off-by: Geert Uytterhoeven <geert@xxxxxxxxxxxxxx>
> > ---
> > Helge: Does this fix the issue for you. I don't have a system with
> >        libnetpbm11 handy yet.
>
> Yes, thanks a lot!
>
> That fixes the build on parisc on Debian 12.0 for me.
> You may add:
> Tested-by: Helge Deller <deller@xxxxxx>
>
> I tried on my Fedora 36/x86_64 as well, and there netpbm-devel
> sadly comes without pkg-config. Nevertheless, based on top of
> your patch by hacking in the include path fixed the issue there
> as well.
> Maybe you can consider applying below patch on top of yours as too?
>
> Helge
> ---------
>
> [PATCH] Add netpbm to include search patch
>
> Netpbm on Fedora 36 comes without pkg-config, so
> manually add the include path.
>
> Signed-off-by: Helge Deller <deller@xxxxxx>

Reviewed-by: Geert Uytterhoeven <geert@xxxxxxxxxxxxxx>
and applying.

Gr{oetje,eeting}s,

                        Geert

-- 
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@xxxxxxxxxxxxxx

In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
                                -- Linus Torvalds



[Index of Archives]     [Video for Linux]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite Tourism]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux