Message ID | 1398061096-14432-1-git-send-email-john.liuli@huawei.com |
---|---|
State | New |
Headers | show |
"john.liuli" <john.liuli@huawei.com> writes: > libfdt_env.h is missing in some stable installs (<1.3.0-3), > some released linux distributions(e.g. Ubuntu 12.04) depending > on it still complain missing libfdt even if already installed. > So give more detailed hint for such OS users. > > Signed-off-by: Liu Li <john.liuli@huawei.com> > --- > configure | 4 +++- > 1 file changed, 3 insertions(+), 1 deletion(-) > > diff --git a/configure b/configure > index 69b9f56..6fe80e0 100755 > --- a/configure > +++ b/configure > @@ -2949,7 +2949,9 @@ EOF > elif test "$fdt" = "yes" ; then > # have neither and want - prompt for system/submodule install > error_exit "DTC (libfdt) not present. Your options:" \ > - " (1) Preferred: Install the DTC (libfdt) devel package" \ > + " (1) Preferred: Install the DTC (libfdt) devel package," \ > + " old version(<1.3.0-3) lost libfdt_env.h, so need" \ "lost"? Do you mean "lacks"? > + " to be fixed manually after installation" \ > " (2) Fetch the DTC submodule, using:" \ > " git submodule update --init dtc" > else If your libfdt-devel package lacks a required header, my advice would be to use the DTC submodule, not to "fix manually" (whatever that means).
On 21 April 2014 07:18, john.liuli <john.liuli@huawei.com> wrote: > libfdt_env.h is missing in some stable installs (<1.3.0-3), > some released linux distributions(e.g. Ubuntu 12.04) depending > on it still complain missing libfdt even if already installed. > So give more detailed hint for such OS users. > > Signed-off-by: Liu Li <john.liuli@huawei.com> > --- > configure | 4 +++- > 1 file changed, 3 insertions(+), 1 deletion(-) > > diff --git a/configure b/configure > index 69b9f56..6fe80e0 100755 > --- a/configure > +++ b/configure > @@ -2949,7 +2949,9 @@ EOF > elif test "$fdt" = "yes" ; then > # have neither and want - prompt for system/submodule install > error_exit "DTC (libfdt) not present. Your options:" \ > - " (1) Preferred: Install the DTC (libfdt) devel package" \ > + " (1) Preferred: Install the DTC (libfdt) devel package," \ > + " old version(<1.3.0-3) lost libfdt_env.h, so need" \ > + " to be fixed manually after installation" \ 1.3.0-3 is an Ubuntu version, not an upstream one, isn't it? We could just update the first line to say "libfdt not present or too old". I agree with Markus that we should definitely not be suggesting that users manually mess with trying to fix up their distro's package. > " (2) Fetch the DTC submodule, using:" \ > " git submodule update --init dtc" > else thanks -- PMM
diff --git a/configure b/configure index 69b9f56..6fe80e0 100755 --- a/configure +++ b/configure @@ -2949,7 +2949,9 @@ EOF elif test "$fdt" = "yes" ; then # have neither and want - prompt for system/submodule install error_exit "DTC (libfdt) not present. Your options:" \ - " (1) Preferred: Install the DTC (libfdt) devel package" \ + " (1) Preferred: Install the DTC (libfdt) devel package," \ + " old version(<1.3.0-3) lost libfdt_env.h, so need" \ + " to be fixed manually after installation" \ " (2) Fetch the DTC submodule, using:" \ " git submodule update --init dtc" else
libfdt_env.h is missing in some stable installs (<1.3.0-3), some released linux distributions(e.g. Ubuntu 12.04) depending on it still complain missing libfdt even if already installed. So give more detailed hint for such OS users. Signed-off-by: Liu Li <john.liuli@huawei.com> --- configure | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-)