jcdickinson/pijul - Change RG4YZPKLWZXRNROBZQD4OLWMF46JPZC4WAM3Q6DBLDQYLL4QNQBQC

file_operations: remove the counter-intuitive --repository options

Created by pmeunier on July 7, 2021

RG4YZPKLWZXRNROBZQD4OLWMF46JPZC4WAM3Q6DBLDQYLL4QNQBQC

Dependencies

In channels

main

Change contents

Deletion in pijul/src/commands/file_operations.rs at line 14 [5.169058]
∅:D[2.96] → [4.2111:2261]
B:BD[5.169248] → [4.2111:2261]
∅:D[4.2261] → [5.169248:169313]
∅:D[6.2581] → [5.169248:169313]
∅:D[7.3663] → [5.169248:169313]
B:BD[5.169248] → [5.169248:169313]
B:BD[5.169313] → [8.32:33]
```
    /// Set the repository where this command should run. Defaults to the first ancestor of the current directory that contains a `.pijul` directory.
    #[clap(long = "repository")]
    repo_path: Option<PathBuf>,
```

Replacement in pijul/src/commands/file_operations.rs at line 25 [5.169058]

B:BD[9.14796] → [3.23739:23806]

        let repo = Repository::find_root(self.repo_path.clone())?;

[9.14796]

[5.169474]

        let repo = Repository::find_root(None)?;

Deletion in pijul/src/commands/file_operations.rs at line 31 [5.169058]
B:BD[5.169598] → [5.169598:169642]
```
        let to = path(&self.repo_path, to);
```

Replacement in pijul/src/commands/file_operations.rs at line 44 [5.169058]

B:BD[5.169973] → [5.169973:170128]

            let source = std::fs::canonicalize(&path(&self.repo_path, p.clone()))?;
            let target = if is_dir { to.join(p) } else { to.clone() };

[5.169973]

[5.170128]

            let source = std::fs::canonicalize(&p.clone())?;
            debug!("source = {:?}", source);
            let target = if is_dir {
                to.join(source.file_name().unwrap())
            } else {
                to.clone()
            };

Insertion in pijul/src/commands/file_operations.rs at line 59 [5.169058]
[10.252]
[11.58]
```
            debug!("target = {:?}", target);
```

Deletion in pijul/src/commands/file_operations.rs at line 87 [5.169058]

∅:D[10.484] → [5.170595:170743]

B:BD[5.170595] → [5.170595:170743]

fn path(root: &Option<PathBuf>, path: PathBuf) -> PathBuf {
    if let Some(ref p) = root {
        p.join(path)
    } else {
        path
    }
}

Deletion in pijul/src/commands/file_operations.rs at line 109 [5.169058]
B:BD[5.171231] → [4.2413:2563]
∅:D[12.330] → [5.171231:171296]
∅:D[4.2563] → [5.171231:171296]
∅:D[6.2881] → [5.171231:171296]
B:BD[5.171231] → [5.171231:171296]
```
    /// Set the repository where this command should run. Defaults to the first ancestor of the current directory that contains a `.pijul` directory.
    #[clap(long = "repository")]
    repo_path: Option<PathBuf>,
```

Replacement in pijul/src/commands/file_operations.rs at line 121 [5.169058]

B:BD[9.15113] → [3.23867:23934]

        let repo = Repository::find_root(self.repo_path.clone())?;

[9.15113]

[13.19543]

        let repo = Repository::find_root(None)?;

Deletion in pijul/src/commands/file_operations.rs at line 179 [5.169058]
B:BD[5.172507] → [5.172507:172540]
B:BD[5.172540] → [4.2564:2714]
∅:D[4.2714] → [5.172540:172572]
∅:D[6.3031] → [5.172540:172572]
∅:D[7.4088] → [5.172540:172572]
B:BD[5.172540] → [5.172540:172572]
```
    #[clap(long = "repository")]
    /// Set the repository where this command should run. Defaults to the first ancestor of the current directory that contains a `.pijul` directory.
    repo_path: Option<PathBuf>,
```

Replacement in pijul/src/commands/file_operations.rs at line 185 [5.169058]

B:BD[9.15285] → [3.23935:24002]

        let repo = Repository::find_root(self.repo_path.clone())?;

[9.15285]

[14.732]

        let repo = Repository::find_root(None)?;

file_operations: remove the counter-intuitive --repository options

Dependencies

In channels

Change contents

Deletion in pijul/src/commands/file_operations.rs at line 14 [5.169058]

Replacement in pijul/src/commands/file_operations.rs at line 25 [5.169058]

Deletion in pijul/src/commands/file_operations.rs at line 31 [5.169058]

Replacement in pijul/src/commands/file_operations.rs at line 44 [5.169058]

Insertion in pijul/src/commands/file_operations.rs at line 59 [5.169058]

Deletion in pijul/src/commands/file_operations.rs at line 87 [5.169058]

Deletion in pijul/src/commands/file_operations.rs at line 109 [5.169058]

Replacement in pijul/src/commands/file_operations.rs at line 121 [5.169058]

Deletion in pijul/src/commands/file_operations.rs at line 179 [5.169058]

Replacement in pijul/src/commands/file_operations.rs at line 185 [5.169058]