-
Notifications
You must be signed in to change notification settings - Fork 1.1k
Update dependecies #5595
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
base: rolling
Are you sure you want to change the base?
Update dependecies #5595
Changes from all commits
c553398
a8e759b
6c648fe
830244a
098a31a
b3f38c6
b9fe431
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,36 +1,49 @@ | ||
alabaster==0.7.12 | ||
Babel==2.14.0 | ||
certifi==2020.6.20 | ||
chardet==4.0.0 | ||
doc8==1.1.1 | ||
docutils==0.20.1 | ||
idna==2.10 | ||
imagesize==1.3.0 | ||
Jinja2==3.0.3 | ||
MarkupSafe==2.0.1 | ||
packaging==21.3 | ||
pbr==5.8.0 | ||
alabaster==0.7.16 | ||
atomicwrites==1.4.1 | ||
attrs==25.3.0 | ||
babel==2.17.0 | ||
certifi==2025.4.26 | ||
charset-normalizer==3.4.2 | ||
click==8.1.8 | ||
codespell==2.4.1 | ||
doc8==1.1.2 | ||
docutils==0.21.2 | ||
idna==3.10 | ||
imagesize==1.4.1 | ||
iniconfig==2.1.0 | ||
Jinja2==3.1.6 | ||
MarkupSafe==3.0.2 | ||
more-itertools==10.7.0 | ||
packaging==25.0 | ||
pbr==6.1.1 | ||
pluggy==1.5.0 | ||
polib==1.2.0 | ||
Pygments==2.17.2 | ||
pyparsing==2.4.7 | ||
py==1.11.0 | ||
Pygments==2.19.1 | ||
pytest==8.3.5 | ||
pytz==2022.1 | ||
requests==2.25.1 | ||
restructuredtext_lint==1.3.2 | ||
snowballstemmer==2.2.0 | ||
Sphinx==7.2.6 | ||
PyYAML==6.0.2 | ||
regex==2024.11.6 | ||
requests==2.32.3 | ||
restructuredtext_lint==1.4.0 | ||
roman-numerals-py==3.1.0 | ||
setuptools==80.3.1 | ||
six==1.17.0 | ||
snowballstemmer==3.0.0.1 | ||
Sphinx==8.2.3 | ||
sphinx-copybutton==0.5.2 | ||
sphinx-lint==0.9.1 | ||
sphinx-lint==1.0.0 | ||
sphinx-multiversion==0.2.4 | ||
sphinx-rtd-theme==2.0.0 | ||
sphinx-tabs==3.4.5 | ||
sphinxcontrib-applehelp==1.0.4 | ||
sphinxcontrib-devhelp==1.0.2 | ||
sphinxcontrib-htmlhelp==2.0.1 | ||
sphinx-rtd-theme==3.0.2 | ||
sphinx-tabs==3.4.7 | ||
sphinx_tamer==0.2.0 | ||
sphinxcontrib-applehelp==2.0.0 | ||
sphinxcontrib-devhelp==2.0.0 | ||
sphinxcontrib-googleanalytics==0.4 | ||
sphinxcontrib-htmlhelp==2.1.0 | ||
sphinxcontrib-jquery==4.1 | ||
sphinxcontrib-jsmath==1.0.1 | ||
sphinxcontrib-mermaid==0.9.2 | ||
sphinxcontrib-qthelp==1.0.3 | ||
sphinxcontrib-serializinghtml==1.1.10 | ||
stevedore==3.5.0 | ||
urllib3==1.26.5 | ||
sphinxcontrib-mermaid==1.0.0 | ||
sphinxcontrib-qthelp==2.0.0 | ||
sphinxcontrib-serializinghtml==2.0.0 | ||
stevedore==5.4.1 | ||
urllib3==2.4.0 |
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -3,7 +3,7 @@ | |
# | ||
# docker build -f docker/image/Dockerfile . | ||
|
||
FROM ubuntu:jammy | ||
FROM ubuntu:noble | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Any plans using a deployed Docker image instead of the Dockerfile? I can create an issue for this if you think this is reasonable to do. |
||
|
||
ARG user=rosindex | ||
ARG uid=1000 | ||
|
@@ -20,7 +20,8 @@ RUN apt-get update && \ | |
graphviz \ | ||
locales \ | ||
make \ | ||
python3-pip && \ | ||
python3-pip \ | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Just to support uv , we could consider uv instead of pip (much faster, venv support, better dep. management). Can create an issue for this if wanted. |
||
python3-venv && \ | ||
rm -rf /var/lib/apt/lists/* | ||
|
||
RUN locale-gen en_US en_US.UTF-8 | ||
|
@@ -32,6 +33,10 @@ ENV HOME=/home/$user | |
ENV PATH=/home/$user/.local/bin:$PATH | ||
WORKDIR /tmp/doc_repository | ||
|
||
USER $user | ||
|
||
CMD ["bash", "-c", "pip3 install --no-warn-script-location --user -r requirements.txt -c constraints.txt && make multiversion"] | ||
RUN python3 -m venv /opt/ros2doc | ||
COPY requirements.txt constraints.txt . | ||
RUN . /opt/ros2doc/bin/activate && pip3 install -r requirements.txt -c constraints.txt | ||
|
||
RUN echo "source /opt/ros2doc/bin/activate" >> /home/$user/.bashrc | ||
USER $user | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Is there any reason that we use a non root user in the container? In the current setup we would need sudo rights for installing pip and apt requirements. |
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -11,3 +11,4 @@ sphinx-rtd-theme | |
sphinx-tabs | ||
sphinxcontrib-mermaid | ||
sphinx-tamer | ||
sphinxcontrib-googleanalytics |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fixed dependencies to latest available versions because they are working with all tools. Or is there a specific reason to stick to the dep availible via apt?