Skip to content
GitLab
  • Menu
Projects Groups Snippets
  • /
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
  • Sign in / Register
  • S seismic-dms-cpp-lib
  • Project information
    • Project information
    • Activity
    • Labels
    • Members
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributors
    • Graph
    • Compare
    • Locked Files
  • Issues 6
    • Issues 6
    • List
    • Boards
    • Service Desk
    • Milestones
    • Iterations
    • Requirements
  • Merge requests 1
    • Merge requests 1
  • CI/CD
    • CI/CD
    • Pipelines
    • Jobs
    • Schedules
    • Test Cases
  • Deployments
    • Deployments
    • Environments
    • Releases
  • Packages & Registries
    • Packages & Registries
    • Package Registry
    • Container Registry
    • Infrastructure Registry
  • Monitor
    • Monitor
    • Incidents
  • Analytics
    • Analytics
    • Value stream
    • CI/CD
    • Code review
    • Insights
    • Issue
    • Repository
  • Wiki
    • Wiki
  • Snippets
    • Snippets
  • Activity
  • Graph
  • Create a new issue
  • Jobs
  • Commits
  • Issue Boards
Collapse sidebar
  • Open Subsurface Data Universe Software
  • Platform
  • Domain Data Mgmt Services
  • Seismic
  • Seismic DMS Suite
  • seismic-dms-cpp-lib
  • Merge requests
  • !85
The source project of this merge request has been removed.

Dockerfiles: semicolon ignores errors

Merged Andras Szalai requested to merge (removed):dockerfiles-semicolon-ignores-errors into master Aug 14, 2021
  • Overview 4
  • Changes 2

Using semicolon in the RUN scripts is a logic error. It ignores all previous errors from the command chain, and continues building an incomplete image.

Interestingly, this was already correct in the third Dockerfile (build.centos7.staticdeps.dev.dockerfile), but not in the other 2.

Assignee
Assign to
Reviewer
Request review from
Time tracking
Source branch: dockerfiles-semicolon-ignores-errors