Re: [blfs-support] LLVM dependency

2021-03-08 Thread Pat Barnes via blfs-support

On 3/8/21 4:30 PM, Pierre Labastie via blfs-support wrote:

On Fri, 2021-03-05 at 12:32 -0500, Pat Barnes via blfs-support wrote:

While installing LLVM in 10.1, I found that Clang now has a
dependency
least when Clang is also installed.


Sorry, a little late on this, but I just built llvm without git on the
computer, and it did not fail.

I think we already have had the case where the build system tries to
use git to download other required dependencies, and fails if git is
not there (I think it was with meson/ninja, it would be the first time
with cmake/ninja). Pat, could you check what git is used for?

Pierre

This is difficult at the moment, since I've since installed all the way 
up to XFCE and Firefox. However, I have my tool that figures out the 
dependecy tree and provides me with the order that packages in BLFS need 
to be built in to install whatever I'm trying to install. Looking out 
the files the order that I installed things, CMake just had been 
installed, so this would have been the first time CMake was used to 
compile a package with ninja.


I don't have the error messages, but they were from CMake and certainly 
could have been downloading something for it, I guess.


Pat

--
http://lists.linuxfromscratch.org/listinfo/blfs-support
FAQ: http://www.linuxfromscratch.org/blfs/faq.html
Unsubscribe: See the above information page


Re: [blfs-support] LLVM dependency

2021-03-08 Thread Joe Locash via blfs-support
On Mon, Mar 8, 2021, 1:24 PM Ken Moffat via blfs-support <
blfs-support@lists.linuxfromscratch.org> wrote:

> On Mon, Mar 08, 2021 at 11:40:08AM -0500, Pat Barnes via
> blfs-support wrote:
> >
> > On 3/5/21 2:54 PM, Ken Moffat via blfs-support wrote:
> > > On Fri, Mar 05, 2021 at 12:32:05PM -0500, Pat Barnes via
> > > blfs-support wrote:
> > > > While installing LLVM in 10.1, I found that Clang now has a
> > > > dependency on GIT. This should be added to the list of
> > > > dependencies for LLVM, at least when Clang is also installed.
> > > >
> > > > Pat
> > > >
> > > Looking at one of my own builds, I can see git is mentioned (and
> > > I install git very early in my build), but I don't think it has
> > > anything to do with clang (although /usr/bin/git-clang-format
> > > does get installed (a python script).
> > >
> > > I can see the following references to GIT in the CMakeLists.txt:
> > >
> > > ./utils/benchmark/src/CMakeLists.txt:"${version_config}"
> > > VERSION ${GIT_VERSION} COMPATIBILITY SameMajorVersion
> > > ./utils/benchmark/CMakeLists.txt:# get_git_version(GIT_VERSION)
> > > ./utils/benchmark/CMakeLists.txt:set(GIT_VERSION "v0.0.0")
> > >
> > > The first of those is:
> > >
> > > include(CMakePackageConfigHelpers)
> > > write_basic_package_version_file( "${version_config}" VERSION
> > > ${GIT_VERSION} COMPATIBILITY SameMajorVersion)
> > >
> > > and the second and third are:
> > >
> > > # Read the git tags to determine the project version # WARNING:
> > > This is meaningless for when the benchmark library is being
> > > built in-tree, # so disable it and hardcode a null version.  #
> > > include(GetGitVersion) # get_git_version(GIT_VERSION)
> > > set(GIT_VERSION "v0.0.0")
> > >
> > > That makes me think it is a piece of boilerplate and that
> > > although cmake looks for it, it doesn't actually use it.
> > >
> > > Or did your build fail, or else not install git-clang-format,
> > > because git was missing ?
> > >
> > > ĸen
> >
> > Yes, my build failed until I installed GIT and then it installed
> > correctly without any problem. The failure occurred after LLVM
> > proper had compiled and the checks for CLANG dependencies egan.
> >
> > Pat
> >
> Thanks for the report.  I've moved git to a required dependency in
> the dev books at r24349 and created errata entries for 10.1 and
> 10.1-systemd.
>

I'm not so sure this should be a required dep.  I build llvm very early
with clang and without git installed and don't get a failure. My build box
doesn't have net access either.  I don't run the checks so it could be that
git is needed for that.

>
-- 
http://lists.linuxfromscratch.org/listinfo/blfs-support
FAQ: http://www.linuxfromscratch.org/blfs/faq.html
Unsubscribe: See the above information page


Re: [blfs-support] LLVM dependency

2021-03-08 Thread Pierre Labastie via blfs-support
On Fri, 2021-03-05 at 12:32 -0500, Pat Barnes via blfs-support wrote:
> While installing LLVM in 10.1, I found that Clang now has a
> dependency 
> least when Clang is also installed.
> 

Sorry, a little late on this, but I just built llvm without git on the
computer, and it did not fail.

I think we already have had the case where the build system tries to
use git to download other required dependencies, and fails if git is
not there (I think it was with meson/ninja, it would be the first time
with cmake/ninja). Pat, could you check what git is used for?

Pierre

-- 
http://lists.linuxfromscratch.org/listinfo/blfs-support
FAQ: http://www.linuxfromscratch.org/blfs/faq.html
Unsubscribe: See the above information page


Re: [blfs-support] LLVM dependency

2021-03-08 Thread Ken Moffat via blfs-support
On Mon, Mar 08, 2021 at 11:40:08AM -0500, Pat Barnes via
blfs-support wrote:
> 
> On 3/5/21 2:54 PM, Ken Moffat via blfs-support wrote:
> > On Fri, Mar 05, 2021 at 12:32:05PM -0500, Pat Barnes via
> > blfs-support wrote:
> > > While installing LLVM in 10.1, I found that Clang now has a
> > > dependency on GIT. This should be added to the list of
> > > dependencies for LLVM, at least when Clang is also installed.
> > > 
> > > Pat
> > > 
> > Looking at one of my own builds, I can see git is mentioned (and
> > I install git very early in my build), but I don't think it has
> > anything to do with clang (although /usr/bin/git-clang-format
> > does get installed (a python script).
> > 
> > I can see the following references to GIT in the CMakeLists.txt:
> > 
> > ./utils/benchmark/src/CMakeLists.txt:"${version_config}"
> > VERSION ${GIT_VERSION} COMPATIBILITY SameMajorVersion
> > ./utils/benchmark/CMakeLists.txt:# get_git_version(GIT_VERSION)
> > ./utils/benchmark/CMakeLists.txt:set(GIT_VERSION "v0.0.0")
> > 
> > The first of those is:
> > 
> > include(CMakePackageConfigHelpers)
> > write_basic_package_version_file( "${version_config}" VERSION
> > ${GIT_VERSION} COMPATIBILITY SameMajorVersion)
> > 
> > and the second and third are:
> > 
> > # Read the git tags to determine the project version # WARNING:
> > This is meaningless for when the benchmark library is being
> > built in-tree, # so disable it and hardcode a null version.  #
> > include(GetGitVersion) # get_git_version(GIT_VERSION)
> > set(GIT_VERSION "v0.0.0")
> > 
> > That makes me think it is a piece of boilerplate and that
> > although cmake looks for it, it doesn't actually use it.
> > 
> > Or did your build fail, or else not install git-clang-format,
> > because git was missing ?
> > 
> > ĸen
> 
> Yes, my build failed until I installed GIT and then it installed
> correctly without any problem. The failure occurred after LLVM
> proper had compiled and the checks for CLANG dependencies egan.
> 
> Pat
> 
Thanks for the report.  I've moved git to a required dependency in
the dev books at r24349 and created errata entries for 10.1 and
10.1-systemd.

ĸen
-- 
RIGHT, he said, PESTILENCE, OPEN ANOTHER PACK OF CARDS. I'M GOING TO GET
TO THE BOTTOM OF THIS IF IT KILLS ME, FIGURATIVELY SPEAKING OF COURSE.
Rincewind grabbed Twoflower and pulled.  -- The Light Fantastic
-- 
http://lists.linuxfromscratch.org/listinfo/blfs-support
FAQ: http://www.linuxfromscratch.org/blfs/faq.html
Unsubscribe: See the above information page


Re: [blfs-support] LLVM dependency

2021-03-08 Thread Pat Barnes via blfs-support


On 3/5/21 2:54 PM, Ken Moffat via blfs-support wrote:

On Fri, Mar 05, 2021 at 12:32:05PM -0500, Pat Barnes via blfs-support wrote:

While installing LLVM in 10.1, I found that Clang now has a dependency on
GIT. This should be added to the list of dependencies for LLVM, at least
when Clang is also installed.

Pat


Looking at one of my own builds, I can see git is mentioned (and I
install git very early in my build), but I don't think it has
anything to do with clang (although /usr/bin/git-clang-format does
get installed (a python script).

I can see the following references to GIT in the CMakeLists.txt:

./utils/benchmark/src/CMakeLists.txt:"${version_config}" VERSION 
${GIT_VERSION} COMPATIBILITY SameMajorVersion
./utils/benchmark/CMakeLists.txt:# get_git_version(GIT_VERSION)
./utils/benchmark/CMakeLists.txt:set(GIT_VERSION "v0.0.0")

The first of those is:

include(CMakePackageConfigHelpers)
write_basic_package_version_file(
 "${version_config}" VERSION ${GIT_VERSION} COMPATIBILITY SameMajorVersion
)

and the second and third are:

# Read the git tags to determine the project version
# WARNING: This is meaningless for when the benchmark library is being built 
in-tree,
# so disable it and hardcode a null version.
# include(GetGitVersion)
# get_git_version(GIT_VERSION)
set(GIT_VERSION "v0.0.0")

That makes me think it is a piece of boilerplate and that although
cmake looks for it, it doesn't actually use it.

Or did your build fail, or else not install git-clang-format,
because git was missing ?

ĸen


Yes, my build failed until I installed GIT and then it installed 
correctly without any problem. The failure occurred after LLVM proper 
had compiled and the checks for CLANG dependencies egan.


Pat


--
http://lists.linuxfromscratch.org/listinfo/blfs-support
FAQ: http://www.linuxfromscratch.org/blfs/faq.html
Unsubscribe: See the above information page