osx/macos hidden files

[lauvergn]

Description

Recently, I've sent a fpm project as a tar file to students. One of them open it with a mac and some hidden files, ._xxx.f90, were created.
Then, "fpm build" fails because fpm try to compile the ._xxx.f90 files.
Of course removing the hidden files solves the problem.

Expected Behaviour

Build the executable without removing the hidden files.

Version of fpm

0.40 (osx) and 0.5.0 (linux)

Platform and Architecture

MacOS, linux

Additional Information

No response

[awvwgk]

Interesting behaviour, I wasn't aware this could happen.

One option for fpm would be to generally ignore all hidden files, the recursive listing of files currently discovers all files ending in f90. I think we can safely assume that projects hiding their source files don't want to compile those, right?

[lauvergn]

"I think we can safely assume that projects hiding their source files don't want to compile those, right?", indeed, I was expected that.

[awvwgk]

The source part of a project are collected with

fpm/src/fpm_filesystem.F90

Line 379 in 2ae0581

recursive subroutine list_files(dir, files, recurse)

which currently picks up all files. Having it skip hidden files and directories would solve this issue.

Alternatively, we can adapt the filter step here.

fpm/src/fpm_sources.f90

Lines 84 to 87 in 2ae0581

	is_source = [(.not.(canon_path(file_names(i)%s) .in. existing_src_files) .and. &
	(str_ends_with(lower(file_names(i)%s), fortran_suffixes) .or. &
	str_ends_with(lower(file_names(i)%s),[".c",".h"]) ),i=1,size(file_names))]
	src_file_names = pack(file_names,is_source)

We should transform the list of source files to their basename and than exclude all hidden files with leading zeros. However, this might be insufficient and still include a hidden directory, i.e. we have to check the canonical path for a pathseparator followed by a dot.

[noisegul]

I'd like to give this issue a try, if it is still free to be assigned.

[awvwgk]

@noisegul sure, feel free to send a patch for this issue. Let us know if you are stuck or need help at any point.

[noisegul]

Will do, thanks.

[noisegul]

@awvwgk Alright, opened a PR.

[awvwgk]

Fixed by #654. Thanks, @noisegul.