Skip to content

fix: change 770 for directory to be able to list it #377

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.

Sign up for GitHub

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

Merged
vrenaville merged 1 commit into from
Nov 28, 2025
Merged

fix: change 770 for directory to be able to list it #377

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
8 changes: 4 additions & 4 deletions 14.0/Dockerfile
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -15,8 +15,8 @@ RUN adduser --disabled-password -u $UID --gid $GID --gecos '' odoo \
&& chown -R odoo:root /var/log/odoo

COPY --chown=odoo:root --chmod=770 ./install /install
COPY --chown=odoo:root --chmod=660 ./base_requirements.txt /odoo
COPY --chown=odoo:root --chmod=660 ./extra_requirements.txt /odoo
COPY --chown=odoo:root --chmod=770 ./base_requirements.txt /odoo
COPY --chown=odoo:root --chmod=770 ./extra_requirements.txt /odoo
Comment on lines +18 to +19
Copy link
Contributor

@florentx florentx Nov 28, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@vrenaville

Why you need the executable bit on requirements files?

I thought it was needed only on the directories.


# Moved because there was a bug while installing `odoo-autodiscover`. There is
# an accent in the contributor name
Expand Down Expand Up @@ -54,9 +54,9 @@ RUN set -x; \
# wait on postgres
RUN /install/dockerize.sh

COPY --chown=odoo:root --chmod=660 ./src_requirements.txt /odoo
COPY --chown=odoo:root --chmod=770 ./src_requirements.txt /odoo
COPY --chown=odoo:root --chmod=770 ./bin /odoo/odoo-bin
COPY --chown=odoo:root --chmod=660 ./templates /templates
COPY --chown=odoo:root --chmod=770 ./templates /templates
COPY --chown=odoo:root --chmod=770 ./before-migrate-entrypoint.d /odoo/before-migrate-entrypoint.d
COPY --chown=odoo:root --chmod=770 ./start-entrypoint.d /odoo/start-entrypoint.d
COPY --chown=odoo:root --chmod=770 ./MANIFEST.in /odoo
Expand Down
8 changes: 4 additions & 4 deletions 15.0/Dockerfile
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -15,8 +15,8 @@ RUN adduser --disabled-password -u $UID --gid $GID --gecos '' odoo \
&& chown -R odoo:root /var/log/odoo

COPY --chown=odoo:root --chmod=770 ./install /install
COPY --chown=odoo:root --chmod=660 ./base_requirements.txt /odoo
COPY --chown=odoo:root --chmod=660 ./extra_requirements.txt /odoo
COPY --chown=odoo:root --chmod=770 ./base_requirements.txt /odoo
COPY --chown=odoo:root --chmod=770 ./extra_requirements.txt /odoo

# Moved because there was a bug while installing `odoo-autodiscover`. There is
# an accent in the contributor name
Expand Down Expand Up @@ -54,9 +54,9 @@ RUN set -x; \
# wait on postgres
RUN /install/dockerize.sh

COPY --chown=odoo:root --chmod=660 ./src_requirements.txt /odoo
COPY --chown=odoo:root --chmod=770 ./src_requirements.txt /odoo
COPY --chown=odoo:root --chmod=770 ./bin /odoo/odoo-bin
COPY --chown=odoo:root --chmod=660 ./templates /templates
COPY --chown=odoo:root --chmod=770 ./templates /templates
COPY --chown=odoo:root --chmod=770 ./before-migrate-entrypoint.d /odoo/before-migrate-entrypoint.d
COPY --chown=odoo:root --chmod=770 ./start-entrypoint.d /odoo/start-entrypoint.d
COPY --chown=odoo:root --chmod=770 ./MANIFEST.in /odoo
Expand Down
8 changes: 4 additions & 4 deletions 16.0/Dockerfile
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -15,8 +15,8 @@ RUN adduser --disabled-password -u $UID --gid $GID --gecos '' odoo \
&& chown -R odoo:root /var/log/odoo

COPY --chown=odoo:root --chmod=770 ./install /install
COPY --chown=odoo:root --chmod=660 ./base_requirements.txt /odoo
COPY --chown=odoo:root --chmod=660 ./extra_requirements.txt /odoo
COPY --chown=odoo:root --chmod=770 ./base_requirements.txt /odoo
COPY --chown=odoo:root --chmod=770 ./extra_requirements.txt /odoo

# Moved because there was a bug while installing `odoo-autodiscover`. There is
# an accent in the contributor name
Expand Down Expand Up @@ -54,9 +54,9 @@ RUN set -x; \
# wait on postgres
RUN /install/dockerize.sh

COPY --chown=odoo:root --chmod=660 ./src_requirements.txt /odoo
COPY --chown=odoo:root --chmod=770 ./src_requirements.txt /odoo
COPY --chown=odoo:root --chmod=770 ./bin /odoo/odoo-bin
COPY --chown=odoo:root --chmod=660 ./templates /templates
COPY --chown=odoo:root --chmod=770 ./templates /templates
COPY --chown=odoo:root --chmod=770 ./before-migrate-entrypoint.d /odoo/before-migrate-entrypoint.d
COPY --chown=odoo:root --chmod=770 ./start-entrypoint.d /odoo/start-entrypoint.d
COPY --chown=odoo:root --chmod=770 ./MANIFEST.in /odoo
Expand Down
8 changes: 4 additions & 4 deletions 17.0/Dockerfile
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -15,8 +15,8 @@ RUN adduser --disabled-password -u $UID --gid $GID --gecos '' odoo \
&& chown -R odoo:root /var/log/odoo

COPY --chown=odoo:root --chmod=770 ./install /install
COPY --chown=odoo:root --chmod=660 ./base_requirements.txt /odoo
COPY --chown=odoo:root --chmod=660 ./extra_requirements.txt /odoo
COPY --chown=odoo:root --chmod=770 ./base_requirements.txt /odoo
COPY --chown=odoo:root --chmod=770 ./extra_requirements.txt /odoo

# Moved because there was a bug while installing `odoo-autodiscover`. There is
# an accent in the contributor name
Expand Down Expand Up @@ -55,9 +55,9 @@ RUN set -x; \
# wait on postgres
RUN /install/dockerize.sh

COPY --chown=odoo:root --chmod=660 ./src_requirements.txt /odoo
COPY --chown=odoo:root --chmod=770 ./src_requirements.txt /odoo
COPY --chown=odoo:root --chmod=770 ./bin /odoo/odoo-bin
COPY --chown=odoo:root --chmod=660 ./templates /templates
COPY --chown=odoo:root --chmod=770 ./templates /templates
COPY --chown=odoo:root --chmod=770 ./before-migrate-entrypoint.d /odoo/before-migrate-entrypoint.d
COPY --chown=odoo:root --chmod=770 ./start-entrypoint.d /odoo/start-entrypoint.d
COPY --chown=odoo:root --chmod=770 ./MANIFEST.in /odoo
Expand Down
8 changes: 4 additions & 4 deletions 18.0/Dockerfile
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -15,8 +15,8 @@ RUN adduser --disabled-password -u $UID --gid $GID --gecos '' odoo \
&& chown -R odoo:root /var/log/odoo

COPY --chown=odoo:root --chmod=770 ./install /install
COPY --chown=odoo:root --chmod=660 ./base_requirements.txt /odoo
COPY --chown=odoo:root --chmod=660 ./extra_requirements.txt /odoo
COPY --chown=odoo:root --chmod=770 ./base_requirements.txt /odoo
COPY --chown=odoo:root --chmod=770 ./extra_requirements.txt /odoo

# Moved because there was a bug while installing `odoo-autodiscover`. There is
# an accent in the contributor name
Expand Down Expand Up @@ -54,9 +54,9 @@ RUN set -x; \
# wait on postgres
RUN /install/dockerize.sh

COPY --chown=odoo:root --chmod=660 ./src_requirements.txt /odoo
COPY --chown=odoo:root --chmod=770 ./src_requirements.txt /odoo
COPY --chown=odoo:root --chmod=770 ./bin /odoo/odoo-bin
COPY --chown=odoo:root --chmod=660 ./templates /templates
COPY --chown=odoo:root --chmod=770 ./templates /templates
COPY --chown=odoo:root --chmod=770 ./before-migrate-entrypoint.d /odoo/before-migrate-entrypoint.d
COPY --chown=odoo:root --chmod=770 ./start-entrypoint.d /odoo/start-entrypoint.d
COPY --chown=odoo:root --chmod=770 ./MANIFEST.in /odoo
Expand Down