Skip to content

Update repository server - #8414

Merged
ev-mp merged 2 commits into
realsenseai:developmentfrom
doronhi:replace_debian_ppa
Feb 23, 2021
Merged

Update repository server#8414
ev-mp merged 2 commits into
realsenseai:developmentfrom
doronhi:replace_debian_ppa

Conversation

@doronhi

@doronhi doronhi commented Feb 22, 2021

Copy link
Copy Markdown
Contributor

replaced realsense-hw-public.s3.amazonaws.com and realsense-hw-publics3-eu-west-1.amazonaws.com addresses with librealsense.intel.com

Installation tested on Ubuntu 16.04, 18.04, 20.04
Tracked on: DSO-16482

….s3-eu-west-1.amazonaws.com addresses with librealsense.intel.com
Comment thread common/model-views.h Outdated
namespace rs2
{
constexpr const char* server_versions_db_url = "http://realsense-hw-public.s3-eu-west-1.amazonaws.com/Releases/rs_versions_db.json";
constexpr const char* server_versions_db_url = "http://librealsense.intel.com/Releases/rs_versions_db.json";

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please revert this change - part of a separate UC

Comment thread CMake/connectivity_check.cmake Outdated
# Copyright(c) 2019 Intel Corporation. All Rights Reserved.
message(STATUS "Checking internet connection...")
file(DOWNLOAD "http://realsense-hw-public.s3-eu-west-1.amazonaws.com/Releases/connectivity_check" "${CMAKE_CURRENT_SOURCE_DIR}/connectivity_check" SHOW_PROGRESS TIMEOUT 5 STATUS status)
file(DOWNLOAD "http://librealsense.intel.com/Releases/connectivity_check" "${CMAKE_CURRENT_SOURCE_DIR}/connectivity_check" SHOW_PROGRESS TIMEOUT 5 STATUS status)

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

https, in this line and in all changes hereafter

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

fixed. Tested on all machines again.

revert changes where the link is a part of a separate UC
@doronhi
doronhi requested a review from ev-mp February 23, 2021 05:15

@ev-mp ev-mp left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good

@ev-mp
ev-mp merged commit f294fba into realsenseai:development Feb 23, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants