rkrp / payasm

A sweet little python assembler which assembles output from dis to python bytecode
GNU General Public License v3.0
4 stars 0 forks source link

Bug with installation #1

Open techvoltage opened 7 years ago

techvoltage commented 7 years ago

python setup.py build: gives error: file '/home/crank/payasm/bin/payasm' does not exist

running install
running bdist_egg
running egg_info
creating payasm.egg-info
writing payasm.egg-info/PKG-INFO
writing top-level names to payasm.egg-info/top_level.txt
writing dependency_links to payasm.egg-info/dependency_links.txt
writing manifest file 'payasm.egg-info/SOURCES.txt'
reading manifest file 'payasm.egg-info/SOURCES.txt'
writing manifest file 'payasm.egg-info/SOURCES.txt'
installing library code to build/bdist.linux-x86_64/egg
running install_lib
running build_py
creating build/bdist.linux-x86_64
creating build/bdist.linux-x86_64/egg
creating build/bdist.linux-x86_64/egg/payasm
copying build/lib.linux-x86_64-2.7/payasm/payasm.py -> build/bdist.linux-x86_64/egg/payasm
copying build/lib.linux-x86_64-2.7/payasm/__init__.py -> build/bdist.linux-x86_64/egg/payasm
byte-compiling build/bdist.linux-x86_64/egg/payasm/payasm.py to payasm.pyc
byte-compiling build/bdist.linux-x86_64/egg/payasm/__init__.py to __init__.pyc
creating build/bdist.linux-x86_64/egg/EGG-INFO
installing scripts to build/bdist.linux-x86_64/egg/EGG-INFO/scripts
running install_scripts
running build_scripts
error: file '/home/crank/payasm/bin/payasm' does not exist
rkrp commented 7 years ago

TL;DR Installation is not possible, at the moment.

The error is because, there is no presence of a command-line script in bin/payasm which exposes the capabilities of the assembler for an end-user usage. I went ahead and added the stub in setup.py post-haste.

I think it is better to concentrate more on building the library for assembling and producing a runnable Python bytecode first. The correctness of the library is ensured using unit tests.

For now, I will add explicitly in the documentation that installation is not possible.

rkrp commented 7 years ago

Depends on fixing #2