Merge lp:~sergiusens/goget-ubuntu-touch/modprobe-orig-files into lp:goget-ubuntu-touch

Proposed by Sergio Schvezov on 2015-09-09
Status: Merged
Approved by: Sergio Schvezov on 2015-09-09
Approved revision: 209
Merged at revision: 208
Proposed branch: lp:~sergiusens/goget-ubuntu-touch/modprobe-orig-files
Merge into: lp:goget-ubuntu-touch
Prerequisite: lp:~sergiusens/goget-ubuntu-touch/align
Diff against target: 25 lines (+8/-0)
1 file modified
ubuntu-device-flash/core.go (+8/-0)
To merge this branch: bzr merge lp:~sergiusens/goget-ubuntu-touch/modprobe-orig-files
Reviewer Review Type Date Requested Status
Sergio Schvezov Approve on 2015-09-09
Michael Vogt 2015-09-09 Approve on 2015-09-09
Review via email: mp+270579@code.launchpad.net

Commit Message

Copy original files from modprobe.d

Description of the Change

I'm not sure we want this (as we can't un-blacklist anything declared here), but at least it is backwards compatible.

To post a comment you must log in.
Michael Vogt (mvo) wrote :

Looks good, thanks for your help on irc understanding this.

review: Approve
Sergio Schvezov (sergiusens) wrote :

approving merge

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'ubuntu-device-flash/core.go'
2--- ubuntu-device-flash/core.go 2015-09-09 14:36:46 +0000
3+++ ubuntu-device-flash/core.go 2015-09-09 19:26:27 +0000
4@@ -17,6 +17,8 @@
5 "path/filepath"
6 "strings"
7 "time"
8+
9+ "launchpad.net/snappy/helpers"
10 )
11
12 // This program is free software: you can redistribute it and/or modify it
13@@ -100,6 +102,12 @@
14 return err
15 }
16
17+ // first we need to copy all the files in modprobe.d
18+ systemModprobeDir := filepath.Join(coreCmd.img.System(), "etc", "modprobe.d")
19+ if err := helpers.RSyncWithDelete(systemModprobeDir, modprobeDir); err != nil {
20+ return err
21+ }
22+
23 modprobeD := filepath.Join(modprobeDir, "ubuntu-core.conf")
24 modprobeDFile, err := os.Create(modprobeD)
25 if err != nil {

Subscribers

People subscribed via source and target branches