<div dir="ltr">Hi,<div><br></div><div>Seems really good, pushed to master</div><div><br></div><div>We can compare each commit in a semi-automated way:</div><div><br></div><div>$ git pull<br></div><div>$ make</div><div>$ cp out/vfs.zip ref.zip</div><div>$ git checkout planescape-torment</div><div>$ git pull (whoops, I wanted to check one commit at time)</div><div>$ git reset --hard 45dadfbe00302d0bfc7c822a7469c5862fa8dcbb</div><div>$ GDP_UNINSTALLED=1 python3 -m tools.check_equivalence planescape-torment | less<br></div><div><br></div><div><div>       "name": "planescape-torment-en-data",</div><div>       "optional": [</div><div>         "autonote.ini",</div><div>         "beast.ini",</div><div>+        "ddrawfix/readme.html",    -> this is likely more related to the original DirectDraw engine</div><div>                                                        than to the game scenario or assets; it doesn't hurt to keep ip tough</div><div>         "keymap.ini",</div><div>         "layout.ini",</div><div>+        "manual.pdf",  --> nice</div><div>         "override/ar0700.ini",</div><div>         "override/ar0901.ini",</div><div>         "quests.ini",</div><div>         "readme.txt",</div><div>+        "readme11.txt",   --> maybe a patch changelog,  locating the original from back then is nice for people</div><div>                                            attempting to use the orignal media and not some already patched GOG/Steam version</div><div>         "torment.ini"</div><div>       ],</div></div><div><br></div><div>Greets,</div><div class="gmail_extra"><br><div class="gmail_quote">2017-09-14 18:44 GMT+02:00 Phil Morrell <span dir="ltr"><<a href="mailto:debian@emorrp1.name" target="_blank">debian@emorrp1.name</a>></span>:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi All,<br>
<br>
I've pushed a "planescape-torment" branch to the repo, please could<br>
you review and merge the changes?<br>
<br>
This is the first time I've needed to create groups to bump the<br>
archive version number, so I'm not confident I did it right - mainly<br>
just followed the baldurs-gate diff.<br>
<br>
Thanks,<br>
--<br>
Phil Morrell (emorrp1)<br>
</blockquote></div><br></div></div>