ValueError seems a little bit user-unfriendly here. What do you think about one of the following?
* Just silently disable mtime preservation when not copying.
* Raise a UserError (see the `beets.ui` package) so a nice error message is printed to the console.