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

Copy custom config files to docker image #208

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

Conversation

hakonhagland
Copy link
Collaborator

Copies a snapshot of LibreOffice config files into the docker image. The config files have been modified from the default LibreOffice config files in two ways:

Copys libreoffice config files into docker image. The config files have
been modifed from the default libreoffice config files in two ways:
First, document comparison mode has been disabled, see PR OPM#204 for
details. Second, the parts sub directory inside the image has been added
as a trusted location to get rid of the popup dialog "Would you like to
change the document, and update all links to get the most recent data?"
@blattms
Copy link
Member

blattms commented Apr 12, 2024

First, document comparison mode has been disabled, see PR #204 for details,

Will this still allow us to compare two different documents (Open the update one, Edit -> Track changes -> Compare Document, and use the old version)? That option is actually quite handy.

@hakonhagland
Copy link
Collaborator Author

@blattms I am not sure why we we would use libreoffice to compare two different documents when we already have git version control? Can you elaborate on that idea?

Anyway, as I remember it, this pull request was in accordance with the README.md file which suggests that we turn off document comparison mode. If the user still want document comparison he could he could avoid using the docker container version of libreoffice or he could manually re-enable the mode from the menu in libreoffice. But it is not recommended if he plans to submit a clean PR..

@blattms
Copy link
Member

blattms commented May 23, 2024

@blattms I am not sure why we we would use libreoffice to compare two different documents when we already have git version control? Can you elaborate on that idea?

Because humans cannot read formatting changes. It is really hard to compare changes using git only. I'll check that myself once I find time.

@hakonhagland
Copy link
Collaborator Author

hakonhagland commented May 23, 2024

Because humans cannot read formatting changes. It is really hard to compare changes using git only. I'll check that myself once I find time.

@blattms Can you check then if it would be a possible strategy for a user to have document comparison mode enabled and only disable it when submitting changes to the git repository? Then we can avoid tracking the revision ids in the xml files which should simplify the git diff review process according to the dicussion about span tags in #203.

@blattms
Copy link
Member

blattms commented May 23, 2024

I will if you tell me how to do this,

@blattms
Copy link
Member

blattms commented May 23, 2024

This is what I am using currently for quick comparison (use "File -> Track Changes -> Compare Documents" and use a copy of the old version):
image

@hakonhagland
Copy link
Collaborator Author

This is what I am using currently for quick comparison (use "File -> Track Changes -> Compare Documents" and use a copy of the old version):

@blattms Thanks for that. What is the problem with that workflow that makes you want to keep the revision ids in the XML files? Sorry, it is not clear to me yet why this workflow is in conflict with this PR.

@blattms
Copy link
Member

blattms commented May 28, 2024

Can you please remind me how to use the docker image. I have been trying this for while now and always fail (e.g. the file `/home/dockeruser/parts/appendices/A.fodt is not found. My approach

  • go to src/python, create venv and pip install .
  • got to docker and do pip install .
  • in the same directory run lodocker-run-container parts/appendices/A.fodt

@hakonhagland
Copy link
Collaborator Author

the file `/home/dockeruser/parts/appendices/A.fodt is not found

I have not tested with pip install . into a venv yet, so there might be some issues here. What I have been using is: poetry install; poetry shell; lodocker-run-container appendices/A.fodt --dir=ubuntu2204-lo7.5.9.

@hakonhagland
Copy link
Collaborator Author

the file `/home/dockeruser/parts/appendices/A.fodt is not found

@blattms Added a fix to the script here: #255. Please check if that works.

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