ezelkow1 commented on a change in pull request #7005:
URL: https://github.com/apache/trafficserver/pull/7005#discussion_r455319637



##########
File path: .vscode/tasks.json
##########
@@ -3,6 +3,20 @@
     // for the documentation about the tasks.json format
     "version": "2.0.0",
     "tasks": [
+        {
+            "label": "Autoreconf",
+            "type": "shell",
+            "command": "autoreconf -if",
+            "problemMatcher": []
+        },
+        {
+            "label": "Configure",
+            "type": "shell",
+            "command": "./configure --prefix=/opt/ats --enable-ccache  
--enable-experimental-plugins --enable-example-plugins --enable-test-tools 
--enable-debug --enable-werror",

Review comment:
       Id be +1 on some env variable for it if possible, we tend to generate 
everything to use /opt/trafficserver here, so having that be customizable would 
be good.  Or maybe just have an optional env var insertion that people could 
throw in stuff like "--prefix=/your/ats/location 
--with-openssl=/your/ssl/location", so you can get custom things you use into 
the build




----------------------------------------------------------------
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.

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


Reply via email to