Opened 5 months ago

Closed 5 months ago

#12192 closed Bug report (fixed)

libfilezilla does not build on case-sensitive filesystems due to incorrect header case in process.cpp

Reported by: Lothar Haeger Owned by:
Priority: normal Component: libfilezilla
Keywords: Cc:
Component version: 0.22.0 Operating system type: OS X
Operating system version: 10.15

Description

Tryig to compile libfilezilla on macOS with case-sensitive APFS file system gives:

libtool: compile:  /usr/bin/clang++ -std=c++17 -DHAVE_CONFIG_H -I. -I../lib/libfilezilla/private -DBUILDING_LIBFILEZILLA -I/opt/local/include/p11-kit-1 -I./libfilezilla -I/opt/local/include -isysroot/Library/Developer/CommandLineTools/SDKs/MacOSX10.15.sdk -fvisibility=hidden -pipe -Os -stdlib=libc++ -isysroot/Library/Developer/CommandLineTools/SDKs/MacOSX10.15.sdk -arch x86_64 -Wall -g -Werror=partial-availability -MT libfilezilla_la-mutex.lo -MD -MP -MF .deps/libfilezilla_la-mutex.Tpo -c mutex.cpp -o libfilezilla_la-mutex.o >/dev/null 2>&1
process.cpp:266:10: fatal error: 'CoreFoundation/CFUrl.h' file not found
#include <CoreFoundation/CFUrl.h>
         ^~~~~~~~~~~~~~~~~~~~~~~~
process.cpp:266:10: note: did not find header 'CFUrl.h' in framework 'CoreFoundation' (loaded from '/Library/Developer/CommandLineTools/SDKs/MacOSX10.15.sdk/System/Library/Frameworks')


Doing the same on a case-insensitive file system works just fine.

Cause: the header file is actually named "CFURL.h" not "CFUrl.h".

PLease find a patch to fix this attached.

Attachments (1)

patch-lib-process.cpp.diff (383 bytes ) - added by Lothar Haeger 5 months ago.
A patch fo fix the error

Download all attachments as: .zip

Change History (2)

by Lothar Haeger, 5 months ago

Attachment: patch-lib-process.cpp.diff added

A patch fo fix the error

comment:1 by Tim Kosse, 5 months ago

Resolution: fixed
Status: newclosed

Thanks, should be fixed now.

Note: See TracTickets for help on using tickets.