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

tpl: modify file system in os functions #13189

Open
wants to merge 23 commits into
base: master
Choose a base branch
from

Conversation

rymut
Copy link

@rymut rymut commented Dec 23, 2024

All os functions now operate under a common file system composed of workdir fs and overlay mount fs with fallback to contentDir for ReadFile, FileExists and Stat function.
This allows to access union file system without introducing new functions and resolves issue with Stat/ReadDir working on different filesystms.

Add the file system access the implementation of: Readdir for rootMappingDir and Size for dirNameOnlyFileInfo.

Fix implementation of collectDirEntries for RootMappingFs to handle root directory correctly.

Releated issues: #12321 #11917 #9604

Access to UFS can be done by partials https://github.com/rymut/hugo-turing/tree/4dea11eb15374325acff1c5203789ee51edec84a/hugo-turing-ufs - but it is cumbersome, so i tried to resolve the issue in hugo code without introducing new parameters or braking chanegs.

rymut added 21 commits December 15, 2024 21:32
All os functions now operate under a common file system composed of
workdir fs and overlay mount fs with fallback to contentDir for
ReadFile, FileExists and Stat function.
This allows to access union file system without introducing new
functions and resolves issue with Stat/ReadDir working on different
filesystms.

Add the file system access the implementation of: Readdir for
rootMappingDir and Size for dirNameOnlyFileInfo.

Fix implementation of collectDirEntries for RootMappingFs to handle
root directory correctly.

Fixes: gohugoio#12321 gohugoio#11917 gohugoio#9604
@CLAassistant
Copy link

CLAassistant commented Dec 23, 2024

CLA assistant check
All committers have signed the CLA.

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.

2 participants