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

Documentation: filesystems_part2: fix typos #381

Merged
merged 1 commit into from
Jan 18, 2024

Conversation

hanyujie2002
Copy link

.. code-block:: c
struct inode_operations minix_dir_inode_operations = {
.create = minix_create,
.lookup = minix_lookup,
.link = minix_link,
.unlink = minix_unlink,
.symlink = minix_symlink,
.mkdir = minix_mkdir,
.rmdir = minix_rmdir,
.mknod = minix_mknod,
//...
};

Creating a link
---------------
The link creation function (hard link) is indicated by the ``symlink`` field in the ``inode_operations`` structure.
In the minix case, the function is :c:func:`minix_link`.

minix_link correspond to link in the first snippet, so the simlink should be modified to link in the second snippet

@hanyujie2002 hanyujie2002 changed the title fix typo: Update filesystems_part2.rst Documentation: filesystems_part2: fix typos Jan 18, 2024
@dbaluta
Copy link
Member

dbaluta commented Jan 18, 2024

Looks like there is a problem building the documentation:


Exception occurred:
  File "/usr/lib/python3/dist-packages/jinja2/sandbox.py", line 350, in is_safe_callable
rending /home/runner/work/linux/linux/Documentation/output/teaching/_images/ditaa-cc9a2e995be74ee99646ea4bf0e551d766fa92ef.png
    return not (getattr(obj, 'unsafe_callable', False) or
jinja2.exceptions.UndefinedError: 'css_tag' is undefined
The full traceback has been saved in /tmp/sphinx-err-k0tebpfp.log, if you want to report the issue to the developers.

Anyone care to investigate?

@hanyujie2002
Copy link
Author

@dbaluta I fixed it in #380

@dbaluta dbaluta merged commit 978a315 into linux-kernel-labs:master Jan 18, 2024
1 check failed
@dbaluta
Copy link
Member

dbaluta commented Jan 18, 2024

@hanyujie2002 thanks!

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