nagios-charmers team mailing list archive

[Chris Sanders <chris.sanders@xxxxxxxxxxxxx>]

Review: Approve

Looks good to me

Diff comments:

> diff --git a/Makefile b/Makefile
> index 9d48829..22955cf 100644
> --- a/Makefile
> +++ b/Makefile
> @@ -1,10 +1,22 @@
>  #!/usr/bin/make
>  PYTHON := /usr/bin/python3
>  export PYTHONPATH := hooks
> +PROJECTPATH = $(dir $(realpath $(MAKEFILE_LIST)))
> +CHARM_NAME = $(notdir $(PROJECTPATH:%/=%))
> +ifndef CHARM_BUILD_DIR
> +    CHARM_BUILD_DIR := /tmp/$(CHARM_NAME)-builds
> +    $(warning Warning CHARM_BUILD_DIR was not set, defaulting to $(CHARM_BUILD_DIR))
> +endif

I don't think we should avoid this functionality due to lack of current standard. If a standard is added later updating this would be easier than adding it fresh.

>  
>  default:
>  	echo Nothing to do
>  
> +build:
> +	@echo "Building charm to base directory $(CHARM_BUILD_DIR)/$(CHARM_NAME)"
> +	@-git describe --tags > ./repo-info
> +	@mkdir -p $(CHARM_BUILD_DIR)/$(CHARM_NAME)
> +	@cp -r * $(CHARM_BUILD_DIR)/$(CHARM_NAME)
> +
>  # Primitive test runner. Someone please fix this.
>  test:
>  	tests/00-setup


-- 
https://code.launchpad.net/~addyess/charm-nagios/+git/charm-nagios/+merge/387090
Your team Nagios Charm developers is subscribed to branch charm-nagios:master.