Bug #2365

The update to the latest Rudder 2.4.0~alpha6~git201203061733 fails at the postinst stage of rudder-webapp

Added by Matthieu CERDA about 3 years ago. Updated 3 months ago.

Status:ReleasedStart date:2012-03-07
Priority:1Due date:
Assignee:Matthieu CERDA% Done:

100%

Category:Packaging
Target version:2.4.0~alpha6
Pull Request: How to reproduce:

Reproduced: Found in version(s):

Description

The update to the latest Rudder 2.4.0~alpha6~git201203061733 fails at the postinst stage of rudder-webapp, because the rudder-upgrade script has a logic error (a return code interpreted as an error code which stops the execution of the script).

Associated revisions

Revision 8ac26130
Added by Matthieu CERDA about 3 years ago

Fixes #2365: The rudder-upgrade script from rudder-webapp has been fixed (failed diff)

History

#1 Updated by Matthieu CERDA about 3 years ago

  • Target version changed from 2.4.0~alpha6 to 2.4.0~alpha7

#2 Updated by Matthieu CERDA about 3 years ago

  • Status changed from In progress to Pending technical review
  • % Done changed from 0 to 100

Applied in changeset commit:8ac2613069a2ab6330bf04015171166ff7ffaf10.

#3 Updated by Jonathan CLARKE about 3 years ago

  • Target version changed from 2.4.0~alpha7 to 2.4.0~alpha6

#4 Updated by Nicolas PERRON about 3 years ago

I find this fix a little odd, but it seems OK.

#5 Updated by Jonathan CLARKE about 3 years ago

  • Status changed from Pending technical review to Released

Nicolas PERRON wrote:

I find this fix a little odd, but it seems OK.

What is odd about it?

#6 Updated by Nicolas PERRON about 3 years ago

Jonathan CLARKE wrote:

Nicolas PERRON wrote:

I find this fix a little odd, but it seems OK.

What is odd about it?

I found this fix odd because of the if statement which doesn't use any "[" but I understand now what was for: handling unexpected exit.

#7 Updated by Nicolas PERRON over 2 years ago

  • Project changed from Rudder to packages
  • Category deleted (11)

#8 Updated by Benoît PECCATTE 3 months ago

  • Project changed from packages to Rudder
  • Category set to Packaging

Also available in: Atom PDF