[PATCH v3 03/13] new: Don't report version after upgrade
Jani Nikula
jani at nikula.org
Sat Aug 23 08:39:09 PDT 2014
On Fri, 01 Aug 2014, Austin Clements <amdragon at MIT.EDU> wrote:
> The version number has always been pretty meaningless to the user and
> it's about to become even more meaningless with the introduction of
> "features". Hopefully, the database will remain on version 3 for some
> time to come; however, the introduction of new features over time in
> version 3 will necessitate upgrades within version 3. It would be
> confusing if we always tell the user they've been "upgraded to version
> 3". If the user wants to know what's new, they should read the news.
I think this is good for now.
What do you think about adding notmuch_database_get_features(), and
printing that?
BR,
Jani.
> ---
> notmuch-new.c | 3 +--
> test/T530-upgrade.sh | 2 +-
> 2 files changed, 2 insertions(+), 3 deletions(-)
>
> diff --git a/notmuch-new.c b/notmuch-new.c
> index d269c7c..b7590a8 100644
> --- a/notmuch-new.c
> +++ b/notmuch-new.c
> @@ -1023,8 +1023,7 @@ notmuch_new_command (notmuch_config_t *config, int argc, char *argv[])
> add_files_state.verbosity >= VERBOSITY_NORMAL ? upgrade_print_progress : NULL,
> &add_files_state);
> if (add_files_state.verbosity >= VERBOSITY_NORMAL)
> - printf ("Your notmuch database has now been upgraded to database format version %u.\n",
> - notmuch_database_get_version (notmuch));
> + printf ("Your notmuch database has now been upgraded.\n");
> }
>
> add_files_state.total_files = 0;
> diff --git a/test/T530-upgrade.sh b/test/T530-upgrade.sh
> index 7d5d5aa..c4c4ac8 100755
> --- a/test/T530-upgrade.sh
> +++ b/test/T530-upgrade.sh
> @@ -33,7 +33,7 @@ test_expect_equal "$output" "\
> Welcome to a new version of notmuch! Your database will now be upgraded.
> This process is safe to interrupt.
> Backing up tags to FILENAME
> -Your notmuch database has now been upgraded to database format version 2.
> +Your notmuch database has now been upgraded.
> No new mail."
>
> test_begin_subtest "tag backup matches pre-upgrade dump"
> --
> 2.0.0
>
> _______________________________________________
> notmuch mailing list
> notmuch at notmuchmail.org
> http://notmuchmail.org/mailman/listinfo/notmuch
More information about the notmuch
mailing list