Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

release.sh: exit -1 must display an ERROR log line #320

Open
sylvaingaudan opened this issue Jun 20, 2018 · 1 comment
Open

release.sh: exit -1 must display an ERROR log line #320

sylvaingaudan opened this issue Jun 20, 2018 · 1 comment
Labels
type:maintenance Minor change, optimization or refactoring

Comments

@sylvaingaudan
Copy link
Member

When release.sh exits with code -1, then it must display an ERROR log line so that the error is quickly identified.

@sylvaingaudan sylvaingaudan added the type:maintenance Minor change, optimization or refactoring label Jun 20, 2018
@sylvaingaudan sylvaingaudan added this to the v7.6.2 milestone Jun 20, 2018
@sylvaingaudan sylvaingaudan self-assigned this Jun 20, 2018
@MohamedHamouGisaia MohamedHamouGisaia modified the milestones: v7.6.2, v7.6.3 Jun 22, 2018
@MohamedHamouGisaia MohamedHamouGisaia modified the milestones: v7.6.3, v7.6.4 Jul 11, 2018
@sfalquier sfalquier modified the milestones: v7.6.4, v7.6.5 Sep 7, 2018
@MohamedHamouGisaia MohamedHamouGisaia modified the milestones: v7.6.5, v8.6.1, v9.6.0 Oct 15, 2018
@MohamedHamouGisaia MohamedHamouGisaia modified the milestones: v9.6.0, v9.6.1 Oct 30, 2018
@elouanKeryell-Even elouanKeryell-Even removed their assignment Nov 16, 2018
@elouanKeryell-Even
Copy link
Contributor

elouanKeryell-Even commented Nov 16, 2018

Something like wrapping each command as follow?

if ! command; then
  >&2 echo "[ERROR] ..."
  exit -1
fi

@MohamedHamouGisaia MohamedHamouGisaia modified the milestones: v9.6.1, v9.6.2 Dec 14, 2018
@MohamedHamouGisaia MohamedHamouGisaia modified the milestones: v9.6.2, v10.6.0 Feb 4, 2019
@MohamedHamouGisaia MohamedHamouGisaia removed this from the v10.6.0 milestone Mar 28, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type:maintenance Minor change, optimization or refactoring
Projects
None yet
Development

No branches or pull requests

4 participants