Skip to content

rename focal_alpine image to alpine_x86_64 #21

Closed

Description

@mattip

The deploy.sh script takes only one argument for the $PLATFORM so in PR #20 that took on the name alpine resulting in a focal_alpine docker tag. It would be nice to refactor this to use a $DISTRO, with a default of focal.

#!/bin/bash
set -ex
docker login -u "$DOCKER_USERNAME" -p "$DOCKER_PASSWORD"
build="multibuild/focal_$PLATFORM"
docker tag ${build}:${TRAVIS_COMMIT} ${build}:latest
docker push ${build}:${TRAVIS_COMMIT}
docker push ${build}:latest

Activity

  1. mattip commented on Oct 13, 2021

    @mattip
    ContributorAuthor
  2. stefanv commented on Oct 13, 2021

    @stefanv
    Contributor

    The distinction between Alpine and Ubuntu seems less the distribution than the libc being used. If you think $DISTRO is the right way to go, I'm happy to make that modification; otherwise $LIBC?

  3. stefanv commented on Oct 13, 2021

    @stefanv
    Contributor

    I'll do the $DISTRO thing for now, and we can refactor later.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions