mirror of
https://github.com/martinvonz/jj.git
synced 2025-05-30 19:32:39 +00:00
It seems better to have the caller pass the transaction description when we finish the transaction than when we start it. That way we have all the information we want to include more readily available.
124 lines
5.1 KiB
Rust
124 lines
5.1 KiB
Rust
// Copyright 2020 The Jujutsu Authors
|
|
//
|
|
// Licensed under the Apache License, Version 2.0 (the "License");
|
|
// you may not use this file except in compliance with the License.
|
|
// You may obtain a copy of the License at
|
|
//
|
|
// https://www.apache.org/licenses/LICENSE-2.0
|
|
//
|
|
// Unless required by applicable law or agreed to in writing, software
|
|
// distributed under the License is distributed on an "AS IS" BASIS,
|
|
// WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
|
|
// See the License for the specific language governing permissions and
|
|
// limitations under the License.
|
|
|
|
use std::io::Write;
|
|
|
|
use jj_lib::backend::ObjectId;
|
|
use jj_lib::rewrite::{merge_commit_trees, restore_tree};
|
|
use tracing::instrument;
|
|
|
|
use crate::cli_util::{user_error, CommandError, CommandHelper, RevisionArg};
|
|
use crate::ui::Ui;
|
|
|
|
/// Restore paths from another revision
|
|
///
|
|
/// That means that the paths get the same content in the destination (`--to`)
|
|
/// as they had in the source (`--from`). This is typically used for undoing
|
|
/// changes to some paths in the working copy (`jj restore <paths>`).
|
|
///
|
|
/// If only one of `--from` or `--to` is specified, the other one defaults to
|
|
/// the working copy.
|
|
///
|
|
/// When neither `--from` nor `--to` is specified, the command restores into the
|
|
/// working copy from its parent(s). `jj restore` without arguments is similar
|
|
/// to `jj abandon`, except that it leaves an empty revision with its
|
|
/// description and other metadata preserved.
|
|
///
|
|
/// See `jj diffedit` if you'd like to restore portions of files rather than
|
|
/// entire files.
|
|
#[derive(clap::Args, Clone, Debug)]
|
|
pub(crate) struct RestoreArgs {
|
|
/// Restore only these paths (instead of all paths)
|
|
#[arg(value_hint = clap::ValueHint::AnyPath)]
|
|
paths: Vec<String>,
|
|
/// Revision to restore from (source)
|
|
#[arg(long)]
|
|
from: Option<RevisionArg>,
|
|
/// Revision to restore into (destination)
|
|
#[arg(long)]
|
|
to: Option<RevisionArg>,
|
|
/// Undo the changes in a revision as compared to the merge of its parents.
|
|
///
|
|
/// This undoes the changes that can be seen with `jj diff -r REVISION`. If
|
|
/// `REVISION` only has a single parent, this option is equivalent to `jj
|
|
/// restore --to REVISION --from REVISION-`.
|
|
///
|
|
/// The default behavior of `jj restore` is equivalent to `jj restore
|
|
/// --changes-in @`.
|
|
#[arg(long, short, value_name="REVISION", conflicts_with_all=["to", "from"])]
|
|
changes_in: Option<RevisionArg>,
|
|
/// Prints an error. DO NOT USE.
|
|
///
|
|
/// If we followed the pattern of `jj diff` and `jj diffedit`, we would use
|
|
/// `--revision` instead of `--changes-in` However, that would make it
|
|
/// likely that someone unfamiliar with this pattern would use `-r` when
|
|
/// they wanted `--from`. This would make a different revision empty, and
|
|
/// the user might not even realize something went wrong.
|
|
#[arg(long, short, hide = true)]
|
|
revision: Option<RevisionArg>,
|
|
}
|
|
|
|
#[instrument(skip_all)]
|
|
pub(crate) fn cmd_restore(
|
|
ui: &mut Ui,
|
|
command: &CommandHelper,
|
|
args: &RestoreArgs,
|
|
) -> Result<(), CommandError> {
|
|
let mut workspace_command = command.workspace_helper(ui)?;
|
|
let (from_tree, to_commit);
|
|
if args.revision.is_some() {
|
|
return Err(user_error(
|
|
"`jj restore` does not have a `--revision`/`-r` option. If you'd like to modify\nthe \
|
|
*current* revision, use `--from`. If you'd like to modify a *different* \
|
|
revision,\nuse `--to` or `--changes-in`.",
|
|
));
|
|
}
|
|
if args.from.is_some() || args.to.is_some() {
|
|
to_commit = workspace_command.resolve_single_rev(args.to.as_deref().unwrap_or("@"), ui)?;
|
|
from_tree = workspace_command
|
|
.resolve_single_rev(args.from.as_deref().unwrap_or("@"), ui)?
|
|
.tree()?;
|
|
} else {
|
|
to_commit =
|
|
workspace_command.resolve_single_rev(args.changes_in.as_deref().unwrap_or("@"), ui)?;
|
|
from_tree = merge_commit_trees(workspace_command.repo().as_ref(), &to_commit.parents())?;
|
|
}
|
|
workspace_command.check_rewritable([&to_commit])?;
|
|
|
|
let matcher = workspace_command.matcher_from_values(&args.paths)?;
|
|
let to_tree = to_commit.tree()?;
|
|
let new_tree_id = restore_tree(&from_tree, &to_tree, matcher.as_ref())?;
|
|
if &new_tree_id == to_commit.tree_id() {
|
|
writeln!(ui.stderr(), "Nothing changed.")?;
|
|
} else {
|
|
let mut tx = workspace_command.start_transaction();
|
|
let mut_repo = tx.mut_repo();
|
|
let new_commit = mut_repo
|
|
.rewrite_commit(command.settings(), &to_commit)
|
|
.set_tree_id(new_tree_id)
|
|
.write()?;
|
|
// rebase_descendants early; otherwise `new_commit` would always have
|
|
// a conflicted change id at this point.
|
|
let num_rebased = tx.mut_repo().rebase_descendants(command.settings())?;
|
|
write!(ui.stderr(), "Created ")?;
|
|
tx.write_commit_summary(ui.stderr_formatter().as_mut(), &new_commit)?;
|
|
writeln!(ui.stderr())?;
|
|
if num_rebased > 0 {
|
|
writeln!(ui.stderr(), "Rebased {num_rebased} descendant commits")?;
|
|
}
|
|
tx.finish(ui, format!("restore into commit {}", to_commit.id().hex()))?;
|
|
}
|
|
Ok(())
|
|
}
|