Skip to content

Fix/cleanup Makefile#1

Open
fuhry wants to merge 1 commit into
dolow:mainfrom
fuhry:fix-makefile
Open

Fix/cleanup Makefile#1
fuhry wants to merge 1 commit into
dolow:mainfrom
fuhry:fix-makefile

Conversation

@fuhry

@fuhry fuhry commented Oct 18, 2022

Copy link
Copy Markdown

I encountered some build errors/warnings on my system as a result of building the project outside of the original source location. The following improvements are made to the Makefile:

  • The "c" target now runs phpize and configure
  • Added an all target that builds cgo and c
  • Separated installation out into an install target
  • Added a clean target to delete build artifacts
  • Added /c/main.dep and /c/Makefile to .gitignore

I encountered some build errors/warnings on my system as a result of building the project outside of the original source location. The following improvements are made to the Makefile:

- The "c" target now runs `phpize` and `configure`
- Added an `all` target that builds `cgo` and `c`
- Separated installation out into an `install` target
- Added a `clean` target to delete build artifacts
- Added `/c/main.dep` and `/c/Makefile` to `.gitignore`
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