emojicode / emojicode

😀😜🔂 World’s only programming language that’s bursting with emojis

Home Page:https://emojicode.org

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

Please install the required when the 'install' target is called

yurivict opened this issue · comments

It currently doesn't install anything.

install.sh wouldn't do because it doesn't do the right things:

  1. It uses sudo that isn't needed to install the project
  2. It expects /usr/local/bin to be writable which isn't what should be done to install

The install target should install into $DESTDIR$PREFIX and should never need to use sudo.

commented

install.sh doesn’t require sudo, it offers to use it if it cannot write into the installation directories.

The directories in which to install are fully customizable. The installer states this every time it is run:

If you prefer different locations you can rerun me and provide me with other locations like so:
	 ./install.sh [binary location] [packages location] [include location]

The installation process should be fully automatic. There's nobody there to read what the installaion script states.

commented

Feel free to open a pull request that makes the necessary changes for this to run automatically (if possible in terms of calling install.sh, this is what I wanted to hint at).

Sorry, I don't have resources. I am only a package maintainer.

Project authors are expected to make projects installable, otherwise users would simply have no chance to even see your software because most people install software using package managers these days.

I don't see anything the install.sh script that CMake's install() function can't do, and better.