Bug #7691

Sometimes fails to build due to 40-include_syslinux_in_ISO_filesystem

Added by intrigeri 2014-07-31 08:19:00 . Updated 2014-09-02 05:15:58 .

Status:
Resolved
Priority:
Elevated
Assignee:
Category:
Build system
Target version:
Start date:
2014-07-31
Due date:
% Done:

100%

Feature Branch:
bugfix/7691-fix-ftbfs-with-apt-get-source-syslinux
Type of work:
Code
Blueprint:

Starter:
Affected tool:
Deliverable for:

Description

As reported by various people on IRC and tails-dev, our current Git trees fail to build under certain circumstances in 40-include_syslinux_in_ISO_filesystem@. Indeed, it’s even surprising that this hook works for me.


Subtasks


Related issues

Related to Tails - Bug #7661: Building an ISO as documented can fail in many ways Resolved 2014-07-25

History

#1 Updated by intrigeri 2014-07-31 08:22:32

  • related to Bug #7661: Building an ISO as documented can fail in many ways added

#2 Updated by intrigeri 2014-07-31 08:31:04

  • Feature Branch set to bugfix/7691-fix-ftbfs-with-apt-get-source-syslinux
  • Type of work changed from Communicate to Code

#3 Updated by intrigeri 2014-07-31 09:45:15

  • Assignee deleted (intrigeri)
  • % Done changed from 10 to 50
  • QA Check set to Ready for QA

#4 Updated by intrigeri 2014-08-03 17:13:49

  • Assignee set to alant

#5 Updated by intrigeri 2014-08-11 13:49:06

  • Priority changed from Normal to Elevated

#6 Updated by alant 2014-08-14 09:47:42

  • Status changed from In Progress to Fix committed
  • Assignee deleted (alant)
  • % Done changed from 50 to 100
  • QA Check changed from Ready for QA to Pass

The code and APT review passes, but:

- I don’t know how to reproduce the bug that this is supposed to fix. I
thus trust Kill Your TV’s test
- the branch was actually already merged in stable by 554c3c3 “Merge
remote-tracking branch ‘kytv/feature/i2p-bootparam’ into stable”
because that branch was based on the one I’m supposed to merge. Dear
reviewer of kytv/feature/i2p-bootparam, please check twice what you
merge next time!

Anyway, it’s merged and I see no reason to unmerge it.

#7 Updated by anonym 2014-09-02 05:15:58

  • Status changed from Fix committed to Resolved