<html><head><style>pre,code,address {
margin: 0px;
}
h1,h2,h3,h4,h5,h6 {
margin-top: 0.2em;
margin-bottom: 0.2em;
}
ol,ul {
margin-top: 0em;
margin-bottom: 0em;
}
blockquote {
margin-top: 0em;
margin-bottom: 0em;
}
</style></head><body><div>I think it would be good if you, instead of importing the code via plain dump in [1], based your github repo on the upstream one ([2]). This will preserve the existing history, and is likely to make pulls from your repo easier.</div><div><br></div><div>Similarly, you should strive to preserve the authorship of your applied commits. As it stands, the listed author and dates of the changes (e.g. the ones from me) in your repo are incorrect.</div><div><br></div><div>From the archives, this is most easily done by downloading the relevant monthly archive file from [3], and (after unpacking) run e.g. `git am -i 2023-August.txt`. This will ask you for each email whether or not to try to apply the patch.</div><div><br></div><div>If the mail list was noisier, making iteration through all messages inconvenient, you can first split the archive file into individual emails with `mkdir 2023-aug; git mailsplit -o2023-aug -- 2023-August.txt`, then use `git am` on individual files in the extracted folder.</div><div><br></div><div>From the bug tracker, posted patch files should be possible to apply the same way. Posted snippets are a bit iffier, since they don't necessarily come with a commit message or other metadata. For proper attribution, I'd recommend adding a `Suggested-by:` tag at the bottom of the commit message, in addition to the issue link (which you seem to have done a good job of including).</div><div><br></div><div>[1] <a href="https://github.com/poddmo/abcde/commit/6e8d7b65e9e8fa72cdd44b3c781e72a623e71bc8">https://github.com/poddmo/abcde/commit/6e8d7b65e9e8fa72cdd44b3c781e72a623e71bc8</a></div><div>[2] <a href="https://git.einval.com/cgi-bin/gitweb.cgi?p=abcde.git">https://git.einval.com/cgi-bin/gitweb.cgi?p=abcde.git</a></div><div>[3] <a href="https://lists.einval.com/pipermail/abcde-users/">https://lists.einval.com/pipermail/abcde-users/</a></div><div><br></div><div>//Snild</div><div><br></div><div>On Sun, 2024-07-07 at 14:02 +1000, josh wrote:</div><blockquote type="cite" style="margin:0 0 0 .8ex; border-left:2px #729fcf solid;padding-left:1ex"><div dir="ltr">I've ripped hundreds of CD's with abcde and before I embark on the next few hundred, I wanted to fix some bugs and have a go at some enhancements. I've been doing this over the last few months and I've posted my changes to github:<div><br></div><div><a href="https://github.com/poddmo/abcde">https://github.com/poddmo/abcde</a><br></div><div><br></div><div>The bug fixes come from patches attached to bugzilla or posted to the mailing list. I tried for a while to keep track of what came from where but my github commit log is the best reference for changes: <a href="https://github.com/poddmo/abcde/commits/main/">https://github.com/poddmo/abcde/commits/main/</a></div><div><br></div><div>I've now tagged this version as 2.10-RC1 to recognise all the changes. The main new features are:</div><div><ul><li>cddb choices show the year of release</li><li>cddb choice has a help and re-list option. The help also has details about how to diff two releases</li><li>Improved handling of CD sets. </li><ul><li>The -W option is expanded to optionally also take the total number of discs in the set. That number is used for a DISCTOTAL FLAC tag</li><li>If DISCNUMBER is greater than 1, default to append playlist rather than erase</li></ul><li>Define a local file and directory for album art. If the album artwork download fails or the user chooses to overide the downloaded art, abcde will look for ALBUMARTDIR/ALBUMARTFILE before asking the user to manually enter the filename.</li><li style="box-sizing:border-box;margin-top:0.25em">Created munge helper functions:<ul dir="auto" style="box-sizing:border-box;padding-left:2em;margin-top:0px;margin-bottom:0px"><li style="box-sizing:border-box">munge_fs-safe: remove characters that are not Windows and Linux file-system safe or sane</li><li style="box-sizing:border-box;margin-top:0.25em">munge_Startcase: capitalise the first letter of every word</li><li style="box-sizing:border-box;margin-top:0.25em">munge_unicodetoascii: transliterate Unicode to ASCII</li><li style="box-sizing:border-box;margin-top:0.25em">munge_collapsewhitespace: replace multiple spaces with a single space</li></ul></li><li style="box-sizing:border-box;margin-top:0.25em">Add new mungefilename config example to use munge helper functions</li><li style="box-sizing:border-box;margin-top:0.25em">Create new config option STARTCASEFLACTAGS to use munge_Startcase for FLAC tags artist name, album title and track titles</li><ul dir="auto" style="box-sizing:border-box;padding-left:2em;margin-top:0px;margin-bottom:0px"><li style="box-sizing:border-box">This helps to prevent artist duplication due to inconsistent capitalisation</li></ul></ul><div>I'll now test this version with a few tens of CDs. I'm only converting to FLAC so I would appreciate any feedback if you use other formats. Please report issues to the github page with the command line, your config file and details of the audio CD that you're ripping.</div></div></div><div>_______________________________________________<br></div><div>Abcde-users mailing list<br></div><div><a href="mailto:Abcde-users@lists.einval.com">Abcde-users@lists.einval.com</a><br></div><div><a href="https://lists.einval.com/cgi-bin/mailman/listinfo/abcde-users">https://lists.einval.com/cgi-bin/mailman/listinfo/abcde-users</a><br></div></blockquote><div><br></div><div><span></span></div></body></html>