[
https://issues.apache.org/jira/browse/THRIFT-4452?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16323909#comment-16323909
]
ASF GitHub Bot commented on THRIFT-4452:
----------------------------------------
Github user jeking3 commented on a diff in the pull request:
https://github.com/apache/thrift/pull/1464#discussion_r161215006
--- Diff: build/docker/ubuntu-artful/Dockerfile ---
@@ -41,10 +43,21 @@ RUN apt-get update && \
# dotnet (core)
RUN curl https://packages.microsoft.com/keys/microsoft.asc | gpg --dearmor
> /etc/apt/trusted.gpg.d/microsoft.gpg && \
- echo "deb [arch=amd64]
https://packages.microsoft.com/repos/microsoft-ubuntu-artful-prod artful main"
> /etc/apt/sources.list.d/dotnetdev.list
+ echo "deb [arch=amd64]
https://packages.microsoft.com/repos/microsoft-ubuntu-artful-prod artful main"
> \
+ /etc/apt/sources.list.d/dotnetdev.list
-# node.js (this step runs apt-get update internally) - if we ever want a
later version
-RUN curl -sL https://deb.nodesource.com/setup_8.x | bash -
+# node.js
+RUN curl -sL https://deb.nodesource.com/gpgkey/nodesource.gpg.key | sudo
apt-key add - \
+ echo "deb https://deb.nodesource.com/node_8.x artful main" | sudo tee
/etc/apt/sources.list.d/nodesource.list
+# D
+RUN apt-key adv --keyserver keyserver.ubuntu.com --recv-keys
EBCF975E5BA24D5E && \
+ wget
http://master.dl.sourceforge.net/project/d-apt/files/d-apt.list -O
/etc/apt/sources.list.d/d-apt.list && \
+ wget -qO - https://dlang.org/d-keyring.gpg | apt-key add -
+
+# Dart
+RUN curl https://dl-ssl.google.com/linux/linux_signing_key.pub | apt-key
add - && \
+ curl
https://storage.googleapis.com/download.dartlang.org/linux/debian/dart_stable.list
> \
+ /etc/apt/sources.list.d/dart_stable.list
--- End diff --
You need to run apt-get update here after adding these repositories.
> optimize Dockerfile (only onetime apt-get update)
> -------------------------------------------------
>
> Key: THRIFT-4452
> URL: https://issues.apache.org/jira/browse/THRIFT-4452
> Project: Thrift
> Issue Type: Improvement
> Components: Build Process
> Affects Versions: 0.11.0
> Reporter: Robert Lu
> Assignee: Robert Lu
> Priority: Minor
>
> When build
> [apache/thrift-build|https://github.com/apache/thrift/blob/a15060a/build/docker/ubuntu-trusty/Dockerfile],
> there are many times to run apt-get update.
> We can just run one time apt-get update after add all repo.
--
This message was sent by Atlassian JIRA
(v6.4.14#64029)
