Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Commit ec8c48a

Browse files
ByronByron
committed
rename verify-pack to pack-verify (keeping it more formal)
'Formal' in the sense that this creates a hierarchy of nouns on which to perform an action, sorting things alphabetically will work better that way and it will help eventual command-line completions.
1 parent d3c00c8 commit ec8c48a

11 files changed

Lines changed: 29 additions & 29 deletions

‎Makefile‎

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -102,10 +102,10 @@ $(linux_repo):
102102

103103
stress: ## Run various algorithms on big repositories
104104
$(MAKE) -j3 $(linux_repo) $(rust_repo) release-lean
105-
time ./target/release/gixp verify-pack --verbose --statistics $(rust_repo)/.git/objects/pack/*.idx
106-
time ./target/release/gixp verify-pack --verbose --algorithm less-memory $(rust_repo)/.git/objects/pack/*.idx
107-
time ./target/release/gixp verify-pack --verbose --re-encode $(rust_repo)/.git/objects/pack/*.idx
108-
time ./target/release/gixp verify-pack --verbose --re-encode $(linux_repo)/objects/pack/*.idx
105+
time ./target/release/gixp pack-verify --verbose --statistics $(rust_repo)/.git/objects/pack/*.idx
106+
time ./target/release/gixp pack-verify --verbose --algorithm less-memory $(rust_repo)/.git/objects/pack/*.idx
107+
time ./target/release/gixp pack-verify --verbose --re-encode $(rust_repo)/.git/objects/pack/*.idx
108+
time ./target/release/gixp pack-verify --verbose --re-encode $(linux_repo)/objects/pack/*.idx
109109

110110
##@ Maintenance
111111

‎git-odb/tests/pack/file.rs‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -26,7 +26,7 @@ mod method {
2626
}
2727
}
2828

29-
/// All hardcoded offsets are obtained via `git verify-pack --verbose tests/fixtures/packs/pack-a2bf8e71d8c18879e499335762dd95119d93d9f1.idx`
29+
/// All hardcoded offsets are obtained via `git pack-verify --verbose tests/fixtures/packs/pack-a2bf8e71d8c18879e499335762dd95119d93d9f1.idx`
3030
mod decode_entry {
3131
use crate::{fixture_path, fixup, pack::file::pack_at, pack::SMALL_PACK};
3232
use bstr::ByteSlice;

‎src/plumbing/lean.rs‎

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -24,13 +24,13 @@ mod options {
2424
#[derive(FromArgs, PartialEq, Debug)]
2525
#[argh(subcommand)]
2626
pub enum SubCommands {
27-
VerifyPack(VerifyPack),
27+
PackVerify(PackVerify),
2828
}
2929

3030
/// Initialize the repository in the current directory.
3131
#[derive(FromArgs, PartialEq, Debug)]
32-
#[argh(subcommand, name = "verify-pack")]
33-
pub struct VerifyPack {
32+
#[argh(subcommand, name = "pack-verify")]
33+
pub struct PackVerify {
3434
#[argh(switch)]
3535
/// decode and parse tags, commits and trees to validate their correctness beyond hashing correctly.
3636
///
@@ -100,15 +100,15 @@ pub fn main() -> Result<()> {
100100
let cli: Args = crate::shared::from_env();
101101
let thread_limit = cli.threads;
102102
match cli.subcommand {
103-
SubCommands::VerifyPack(VerifyPack {
103+
SubCommands::PackVerify(PackVerify {
104104
path,
105105
verbose,
106106
statistics,
107107
algorithm,
108108
decode,
109109
re_encode,
110110
}) => {
111-
let (_handle, progress) = prepare(verbose, "verify-pack");
111+
let (_handle, progress) = prepare(verbose, "pack-verify");
112112
core::verify::pack_or_pack_index(
113113
path,
114114
progress,

‎src/plumbing/pretty.rs‎

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -28,7 +28,7 @@ mod options {
2828
pub enum Subcommands {
2929
/// Verify the integrity of a pack or index file
3030
#[structopt(setting = AppSettings::ColoredHelp)]
31-
VerifyPack {
31+
PackVerify {
3232
/// output statistical information about the pack
3333
#[structopt(long, short = "s")]
3434
statistics: bool,
@@ -180,7 +180,7 @@ pub fn main() -> Result<()> {
180180
let args = Args::from_args();
181181
let thread_limit = args.threads;
182182
match args.cmd {
183-
Subcommands::VerifyPack {
183+
Subcommands::PackVerify {
184184
path,
185185
algorithm,
186186
verbose,
@@ -191,7 +191,7 @@ pub fn main() -> Result<()> {
191191
progress_keep_open,
192192
statistics,
193193
} => prepare_and_run(
194-
"verify-pack",
194+
"pack-verify",
195195
verbose,
196196
progress,
197197
progress_keep_open,

‎tasks.md‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -18,7 +18,7 @@
1818
* [x] The Rust repo verifies `--decode` and `--re-encode`
1919
* [x] add re-encode to all stress tests
2020
* [x] support easy access to merge-tags and the signature
21-
* **stream pack for verify-pack**
21+
* **stream pack for pack-verify**
2222
* [x] support for multi-threading
2323
* [x] choice of algorithm in pack-verify
2424
* [x] use pack streaming in pack-verify by default

tests/snapshots/plumbing-verify-pack-index-failure renamed to tests/snapshots/plumbing-pack-verify-index-failure

File renamed without changes.

tests/snapshots/plumbing-verify-pack-index-success renamed to tests/snapshots/plumbing-pack-verify-index-success

File renamed without changes.

tests/snapshots/plumbing-verify-pack-index-with-statistics-json-success renamed to tests/snapshots/plumbing-pack-verify-index-with-statistics-json-success

File renamed without changes.

tests/snapshots/plumbing-verify-pack-index-with-statistics-success renamed to tests/snapshots/plumbing-pack-verify-index-with-statistics-success

File renamed without changes.

tests/snapshots/plumbing-verify-pack-success renamed to tests/snapshots/plumbing-pack-verify-success

File renamed without changes.

0 commit comments

Comments
 (0)