big-r81 commented on code in PR #199:
URL: https://github.com/apache/couchdb-pkg/pull/199#discussion_r2830590329


##########
Makefile:
##########
@@ -299,7 +322,7 @@ make-rpmbuild:
 # If we don't change $HOME it'll force building in ~/rpmbuild. Boo.
 build-rpm:
        $(eval HOME := $(shell readlink -f ..))
-       export HOME=$(HOME) && cd ../rpmbuild && rpmbuild --verbose -bb 
SPECS/couchdb.spec --define '_version $(VERSION)'
+       export HOME=$(HOME) && cd ../rpmbuild && rpmbuild --verbose -bb 
SPECS/couchdb.spec --define '_version $(VERSION)' --define '_js_engine 
$(JS_ENGINE)'

Review Comment:
   `SPIDERMONKEY, SPIDERMONKEY_DEV, SM_VER` are replaced via `sed` before, and 
now I see that `SPIDERMONKEY_DEV` isn't set in the spec file at all. Can we 
remove this?



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to