copier icon indicating copy to clipboard operation
copier copied to clipboard

Current working directory during `copier {copy,recopy,update}` should be destination path

Open sisp opened this issue 1 year ago • 2 comments

Describe the problem

While reimplementing jinja-ansible-filters (see #1398) – specifically testing the fileglob filter –, I noticed that the current working directory while rendering template files through copier {copy,recopy,update} is the directory from which the command is executed, but I think it should be the destination path; otherwise, the fileglob filter (and other custom filesystem-based Jinja filters) wouldn't be particularly useful. WDYT?

Template

example.zip

The filesystem layout is as follows:

.
├── cwd.txt
├── dst
│   ├── a.txt
│   └── b.txt
└── src
    └── fileglob.txt.jinja

The example assumes that src/ contains a template from which a subproject will be generated into dst/, which already contains files (e.g., from another Copier template).

To Reproduce

  1. Extract the content of the reproducible example in the ZIP archive above.

  2. Change your current working directory to the root directory where you extracted the archive (i.e. where the file cwd.txt is located).

  3. Run copier copy src/ dst/.

  4. Note the content of the generated file dst/fileglob.txt:

    $ cat dst/fileglob.txt
    ['cwd.txt']
    

    As you can see, the file cwd.txt is listed as a result of {{ '*.txt' | fileglob }}.

Logs

No response

Expected behavior

I'd expect the following behavior:

$ copier copy src/ dst/
$ cat dst/fileglob.txt
['a.txt', 'b.txt']

I.e., filesystem-related Jinja filters should operate relative to the destination path.

That said, filesystem-related Jinja filters operating on the destination path may lead to unexpected behavior because Copier will generate files in arbitrary order, so the output of the fileglob filter may not be consistent across multiple runs.

Screenshots/screencasts/logs

No response

Operating system

Linux

Operating system distribution and version

Ubuntu 22.04

Copier version

9.3.1

Python version

CPython 3.12

Installation method

pipx+pypi

Additional context

No response

sisp avatar Jul 23 '24 12:07 sisp

Something similar happen when you use copier copy ../src/ dst/. After running cd dst ; copier update , ValueError("Local template must be a directory.") is raised because the answers file contains the relative reference, instead of an absolute path.

pcastellazzi avatar Aug 24 '24 01:08 pcastellazzi

That's another subject: https://github.com/copier-org/copier/issues/335

yajo avatar Sep 21 '24 08:09 yajo