firmware-analysis-toolkit icon indicating copy to clipboard operation
firmware-analysis-toolkit copied to clipboard

setup.sh: Consider not patching firmadyne to change interpreter which is standard already

Open Kusekushi opened this issue 2 years ago • 2 comments

https://github.com/attify/firmware-analysis-toolkit/blob/master/setup.sh#L36

Since python2 is EOL anyways (https://www.python.org/doc/sunset-python-2/), it may be worth considering removing this patch to sorta clean-up the firmadyne installation of non-upstream changes.

Kusekushi avatar Dec 21 '22 02:12 Kusekushi

The patch is in place to support Ubuntu 18.04 which defaults to Python2 for the python command. 18.04 goes out of support on April and we should be good to remove the patch then. Besides explicitly referring to Python 3 doesn't harm IMO :)

extremecoders-re avatar Dec 21 '22 09:12 extremecoders-re

Yeah. This is either about removing a patch which isn't really needed after Ubuntu 18.04 goes EOL (and thus making it kinda redundant to leave it in), or keeping the explicitness that already is there. In case there isn't any OS that FAT should support supporting python 2 I'd personally suggest to remove it. Just to save a few lines of code and get that tiny bit closer to plain upstream firmadyne.

I know this isn't doing much of anything at this point right now, though IMHO we should keep as close to plain firmadyne as possible to avoid any possible confusion.

Kusekushi avatar Dec 21 '22 12:12 Kusekushi