Skip to content

Commit

Permalink
Remove reference to <linux/btrfs.h>, and instead use <btrfs/ioctl.h> …
Browse files Browse the repository at this point in the history
…like we're supposed to (from btrfs-progs)

This fixes compilation issues when btrfs.h isn't available (because we just need the relevant structs, which for userspace programs are supposed to come from btrfs-progs instead of the kernel headers).

Docker-DCO-1.1-Signed-off-by: Andrew Page <admwiggin@gmail.com> (github: tianon)
  • Loading branch information
tianon committed Feb 2, 2014
1 parent a446b34 commit 6922f1b
Show file tree
Hide file tree
Showing 3 changed files with 4 additions and 7 deletions.
2 changes: 1 addition & 1 deletion Dockerfile
Original file line number Diff line number Diff line change
Expand Up @@ -32,6 +32,7 @@ RUN apt-get update && DEBIAN_FRONTEND=noninteractive apt-get install -yq \
apt-utils \
aufs-tools \
automake \
btrfs-tools \
build-essential \
curl \
dpkg-sig \
Expand All @@ -40,7 +41,6 @@ RUN apt-get update && DEBIAN_FRONTEND=noninteractive apt-get install -yq \
libapparmor-dev \
libcap-dev \
libsqlite3-dev \
linux-libc-dev \
mercurial \
reprepro \
ruby1.9.1 \
Expand Down
8 changes: 2 additions & 6 deletions graphdriver/btrfs/btrfs.go
Original file line number Diff line number Diff line change
Expand Up @@ -4,15 +4,11 @@ package btrfs

/*
#include <stdlib.h>
#include <sys/ioctl.h>
#include <linux/fs.h>
#include <errno.h>
#include <sys/types.h>
#include <dirent.h>
#include <linux/btrfs.h>
#include <btrfs/ioctl.h>
*/
import "C"

import (
"fmt"
"github.com/dotcloud/docker/graphdriver"
Expand Down
1 change: 1 addition & 0 deletions hack/PACKAGERS.md
Original file line number Diff line number Diff line change
Expand Up @@ -39,6 +39,7 @@ To build docker, you will need the following system dependencies
* Go version 1.2 or later
* SQLite version 3.7.9 or later
* libdevmapper version 1.02.68-cvs (2012-01-26) or later from lvm2 version 2.02.89 or later
* btrfs-progs version 3.8 or later (including commit e5cb128 from 2013-01-07) for the necessary btrfs headers
* A clean checkout of the source must be added to a valid Go [workspace](http://golang.org/doc/code.html#Workspaces)
under the path *src/github.com/dotcloud/docker*.

Expand Down

0 comments on commit 6922f1b

Please sign in to comment.