search

cnvogelg / amitools

Various tools for using AmigaOS programs on other platforms
250 stars 69 forks source link

Commands silently ignored #192

Closed reinauer closed 5 months ago

reinauer commented 7 months ago

I just tried the following command assuming it would patch my ROM image in place: romtool A600.rom patch 1mb_rom

There was no error message or warning, but the file was also not changed. I did not notice this until I burned an EPROM.

The correct way would have been to do this: romtool patch -o nA600.rom A600.rom 1mb_rom

cnvogelg commented 6 months ago

Indeed the "dry-run-by-default" approach is not very user friendly and leads to incorrect usage... Better would be to issue a warning or an error if a mandatory parameter is missing...

cnvogelg commented 6 months ago

Here we go: 95619f9