Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Raise error for same derivatives as source #134

Closed
3 of 9 tasks
gwiedeman opened this issue Apr 15, 2022 · 1 comment
Closed
3 of 9 tasks

Raise error for same derivatives as source #134

gwiedeman opened this issue Apr 15, 2022 · 1 comment
Labels
Core This is part of the main process for creating mailbags Input Parsing input data, such as MBOX, IMAP, PST, EML, etc.
Milestone
early release

Comments

@gwiedeman
Copy link
Collaborator

The problem the component solves

Currently, users can enter the same derivatives format as the source format. for example, mailbagit Inbox.mbox -i mbox -d mbox pdf -m my_mailbag.

This is a bit problematic, as both the source and the derivatives end up in the same format subdirectory. For now, this should raise an error before mailbagit instantiates a controller and does anything. This would only be a problem if/when #28 gets implemented, as I could see a derivative mbox being useful if you are excluding the source mbox. However since #28 is on hold, we should just implement this as simply as possible for now.

Relevant part of mailbag spec?

4.1 Format subdirectories

Type of component

  • Core
  • Input
  • Attachments
  • Derivatives conversion
  • Reporting/Exporting
  • GUI
  • Distribution

Expected contribution

  • Pull Request
  • Comment with proposed solution

Major challenges or things to keep in mind
@gwiedeman gwiedeman added Core This is part of the main process for creating mailbags Input Parsing input data, such as MBOX, IMAP, PST, EML, etc. labels Apr 15, 2022
@gwiedeman gwiedeman added this to the early release milestone Apr 27, 2022
@gwiedeman gwiedeman mentioned this issue Apr 27, 2022
Merged
11 tasks
@gwiedeman
Copy link
Collaborator Author

Addressed by #147

@gwiedeman gwiedeman moved this to Done, merged to develop in Mailbag tool development Aug 22, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Core This is part of the main process for creating mailbags Input Parsing input data, such as MBOX, IMAP, PST, EML, etc.
Projects
Mailbag tool development
Status: Done, merged to develop
Milestone
early release
Development

No branches or pull requests
1 participant
@gwiedeman