Makefile: Remove reference to ART - add some TODO comments

Author: timflannagan

Makefile

@@ -28,15 +28,7 @@ export KUBEBUILDER_ASSETS
 GO := GO111MODULE=on GOFLAGS="$(MOD_FLAGS)" go
 GINKGO := $(GO) run github.com/onsi/ginkgo/ginkgo
 BINDATA := $(GO) run github.com/go-bindata/go-bindata/v3/go-bindata
-
-# ART builds are performed in dist-git, with content (but not commits) copied
-# from the source repo. Thus at build time if your code is inspecting the local
-# git repo it is getting unrelated commits and tags from the dist-git repo,
-# not the source repo.
-# For ART image builds, SOURCE_GIT_COMMIT, SOURCE_GIT_TAG, SOURCE_DATE_EPOCH
-# variables are inserted in Dockerfile to enable recovering the original git
-# metadata at build time.
-GIT_COMMIT := $(if $(SOURCE_GIT_COMMIT),$(SOURCE_GIT_COMMIT),$(shell git rev-parse HEAD))
+GIT_COMMIT := $(shell git rev-parse HEAD))
 
 .PHONY: build test run clean vendor schema-check \
 	vendor-update coverage coverage-html e2e \
@@ -127,6 +119,7 @@ setup-bare: clean e2e.namespace
 	. ./scripts/package_release.sh 1.0.0 test/e2e/resources test/e2e/e2e-bare-values.yaml
 	. ./scripts/install_bare.sh $(shell cat ./e2e.namespace) test/e2e/resources
 
+# TODO(tflannag): Should we remove reference/update e2e to remove OCP namespace?
 # e2e test exculding the rh-operators directory which tests rh-operators and their metric cardinality.
 e2e:
 	go test -v $(MOD_FLAGS)  -failfast -timeout 150m ./test/e2e/... -namespace=openshift-operators -kubeconfig=${KUBECONFIG} -olmNamespace=openshift-operator-lifecycle-manager -dummyImage=bitnami/nginx:latest -ginkgo.flakeAttempts=3
@@ -226,6 +219,8 @@ release: manifests
 
 verify-release: release diff
 
+# TODO(tflannag): Should this target just call a script that can
+# handle these conditionals instead of using make?
 package: olmref=$(shell docker inspect --format='{{index .RepoDigests 0}}' quay.io/operator-framework/olm:v$(ver))
 package:
 ifndef target