Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Ignore hidden folder (e.g. .git folder) #51

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

marcolussetti
Copy link

This addresses #26 which I think is very nice for those who want to keep the photos and metadata in their own Git repository.

@tmountain
Copy link

This should be merged because a .git folder can cause a recursive situation to occur where expose never finishes...

@tmountain
Copy link

I applied this manually and can confirm it fixes the recursion bug when using git.

@krupan
Copy link

krupan commented Jan 29, 2023

I strongly agree this is needed. This might be a better way to do it (puts it in the same place that directories starting wtih _ are handled/ignored):

diff --git a/expose.sh b/expose.sh
index 3aa8d3d..80ac6f5 100755
--- a/expose.sh
+++ b/expose.sh
@@ -239,7 +239,7 @@ do
        nav_name+=("$node_name")
        nav_depth+=("$node_depth")
        nav_type+=("$node_type")
-done < <(find "$topdir" -type d ! -path "$topdir*/_*" | sort)
+done < <(find "$topdir" -type d ! -path "$topdir*/_*" ! -path "$topdir*/.*" | sort)
 
 # re-create directory structure
 mkdir -p "$topdir/_site"

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants