Skip to content

feat(build_fpc): proposing the following modifications to the script v - #157

Open
megapixel-code wants to merge 1 commit into
genericptr:trunkfrom
megapixel-code:script_update
Open

feat(build_fpc): proposing the following modifications to the script v#157
megapixel-code wants to merge 1 commit into
genericptr:trunkfrom
megapixel-code:script_update

Conversation

@megapixel-code

Copy link
Copy Markdown
  • Listen to the env variable FPC_CFG
  • use !/usr/bin/env bash instead of !/bin/bash in the shebang (not very important)

Hi, I am trying to package this program to NixPkgs. To be able to do so I need to be able to set a custom path for the FPC_CFG variable. fpc.cfg is not located in the same directory as the fpc program in nix.

also the use of the shebang !/usr/bin/env bash is preferable to !/bin/bash because not all UNIX system have a /bin/bash (nixos doesnt have it for example).

this is not very important for me as i can just ignore the shebang by executing the program like bash ./src/build_fpc.sh

- Listen to the env variable FPC_CFG
- use !/usr/bin/env bash instead of !/bin/bash in the shebang (not very important)
Comment thread src/build_fpc.sh
#=============================================================================

# Global: will be set by detect_fpc
FPC_CFG=""

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Not needed because bash default value is an empty string

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant