novoid / move2archive

Managing event-related files in a folder hierarchy like "<ARCHIVE>/2013/2013-05-17 Event name/"
GNU General Public License v3.0
23 stars 4 forks source link

shorter default path to archive #4

Closed nbehrnd closed 2 years ago

nbehrnd commented 2 years ago

By default, earlier commits of m2a assumed the eventual deposit of files in

~/archive/events_memories

i.e. with ~/archive containing only one folder, events_memories.

Though m2a allows to adjust this by --archivepath, the direct move into ~/archive (without the extra level of hierarchy) appears just as functional and more compact.

novoid commented 2 years ago

Hi, I understand your goal and I agree that for the generic use-case, the default does seem to be questionable. However, it reflects my personal situation as you can read on https://karl-voit.at/folder-hierarchy/

So, I'll decline this pull request as it is.

But: I'd propose a small fix instead: if the "archive/events_memory" is found, the default is set to this. If not, the default is assumed to be "archive". I guess this does qualify as a good compromise, right?

novoid commented 2 years ago

See https://github.com/novoid/move2archive/commit/d40214521a9aea25eb0faf4df101aca414048496

nbehrnd commented 2 years ago

Karl Voit @.***> wrote:

But: I'd propose a small fix instead: if the "archive/events_memory" is found, the default is set to this. If not, the default is assumed to be "archive". I guess this does qualify as a good compromise, right?

Yes, the compromise does qualify.