[Bug bootstrap/95005] zstd.h not found if installed in non-system prefix

2020-05-20 Thread gcc at ikkoku dot de
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=95005

--- Comment #7 from Michael Kuhn  ---
Took me a while, sorry. I have just sent the patch to the list.

[Bug bootstrap/95005] zstd.h not found if installed in non-system prefix

2020-05-12 Thread gcc at ikkoku dot de
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=95005

Michael Kuhn  changed:

   What|Removed |Added

  Attachment #48482|0   |1
is obsolete||

--- Comment #5 from Michael Kuhn  ---
Created attachment 48516
  --> https://gcc.gnu.org/bugzilla/attachment.cgi?id=48516=edit
Updated zstd patch

Thanks for investigating! Sadly, this still does not work for me, even with
master.
If I do "make lto-compress.o" in the gcc subdirectory, it works (and includes
the include path), but a normal "make" in the build directory does not.

In Spack, another issue has come up: When building with Clang (as frequently
done on macOS), libzstd cannot be found. After some investigation, it seems
that the library path is also not set correctly.

I have come up with the attached updated patch that allows me build GCC with
zstd in a non-standard path. It is also a bit cleaner than the previous
approach.

[Bug bootstrap/95005] zstd.h not found if installed in non-system prefix

2020-05-11 Thread gcc at ikkoku dot de
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=95005

--- Comment #2 from Michael Kuhn  ---
(In reply to Martin Liška from comment #1)
> Thank you for the bug report.
> I'm sending a better patch, can you please test it?

I applied your patch on top of the GCC 10.1.0 tarball. Sadly, it still fails
with the same error. It looks like ZSTD_CPPFLAGS is not actually exported/set
anywhere (only available in configure), which is why I made use of
ZSTD_INCLUDE.

[Bug other/95005] New: zstd.h not found if installed in non-system prefix

2020-05-08 Thread gcc at ikkoku dot de
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=95005

Bug ID: 95005
   Summary: zstd.h not found if installed in non-system prefix
   Product: gcc
   Version: 10.1.0
Status: UNCONFIRMED
  Severity: normal
  Priority: P3
 Component: other
  Assignee: unassigned at gcc dot gnu.org
  Reporter: gcc at ikkoku dot de
  Target Milestone: ---

Created attachment 48482
  --> https://gcc.gnu.org/bugzilla/attachment.cgi?id=48482=edit
Add zstd include directory

While adding GCC 10.1.0 to Spack, I ran into a problem with zstd: When
specifying a non-system prefix with --with-zstd (Spack installs each package in
its own prefix), the build of lto-compress.o failed because it could not find
the zstd.h header.

The attached patch fixes the problem. It should probably be changed to export
the appropriate compiler flag from configure.ac to avoid the ifeq in the
Makefile. However, I wanted to avoid touching configure.ac so we do not have to
regenerate configure.