Skip to content

GitLab

  • Menu
Projects Groups Snippets
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
  • Sign in / Register
  • slapos slapos
  • Project information
    • Project information
    • Activity
    • Labels
    • Members
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributors
    • Graph
    • Compare
  • Merge requests 122
    • Merge requests 122
  • CI/CD
    • CI/CD
    • Pipelines
    • Jobs
    • Schedules
  • Deployments
    • Deployments
    • Environments
    • Releases
  • Analytics
    • Analytics
    • Value stream
    • CI/CD
    • Repository
  • Snippets
    • Snippets
  • Activity
  • Graph
  • Jobs
  • Commits
Collapse sidebar
  • nexedi
  • slaposslapos
  • Merge requests
  • !430

Merged
Created Oct 29, 2018 by Kirill Smelkov@kirrOwner

golang: Don't require users to put \ into gowork.install

  • Overview 6
  • Commits 1
  • Changes 3

Previously it was required to put \ into multi-entry gowork.install, because the way it was processed was to directly pass it into bash command, and if \n were there the command was cut by bash and the trailing was considered as another command.

We can avoid that by explicitly processing \n in gowork.install and translate it to just ' '. This helps people because now gowork.install becomes just a regular multiline buildout entry, similar e.g. to gowork.cpkgpath et all.

Here is one example where it helps:

https://lab.nexedi.com/jerome/slapos/commit/95c45fe0#note_68408

Assignee
Assign to
Reviewer
Request review from
Time tracking
Source branch: y/golang-install-trailshash
GitLab Nexedi Edition | About GitLab | About Nexedi | 沪ICP备2021021310号-2 | 沪ICP备2021021310号-7