On 2/4/20 6:48 PM, Ken Moffat via blfs-dev wrote:
My first attempt to build icewm-1.6.4, but it failed early on (and
the wonderful cmake kept churning on the other cores until the Error
was way in the past and I thought it had failed silently until I
tried to continue in the existing source.

Unfortunately, icewm has TWO issue trackers - the issue showed up in
the second one, https://github.com/bbidulock/icewm/issues/403
but a comment in that said it was fixed at the primary issue
tracker, https://github.com/bbidulock/icewm/issues/403 where Bruce
said that it didn't fix it for him, but that enabling LTO did fix
it.

Bruce, does what is in the book build for you ?

Oops. No, but this does:

sed -i '/ADD_EXECUTABLE(icesh/s/yarray.cc/& ytimer.cc/' src/CMakeLists.txt &&

    mkdir build &&
    cd    build &&

    cmake -DCMAKE_INSTALL_PREFIX=/usr \
          -DCMAKE_BUILD_TYPE=Release  \
          -DCFGDIR=/etc               \
          -DENABLE_LTO=ON             \
          -DCMAKE_EXE_LINKER_FLAGS='-lXrandr -lXinerama' \
          -DDOCDIR=/usr/share/doc/$PROGRAM \
          ..
    make -j4 &&
    cp ../lib/IceWM.jpg lib &&

    $SUDO make install         &&
    $SUDO rm /usr/share/xsessions/icewm.desktop

For me, adding the following patch fixes it for me (on top of what
is in the book, so using LTO).  I couldn't persuade sed to play ball
with the ')' for a one-liner at the ')$' part of the line, with
single escapes it complained one or more were unmatched, with double
escapes it was happy but changed nothing, so for the moment I'm
using this patch (needs proper heading and naming unless someone can
do it with sed).

ĸen

--- icewm-1.6.4/src/CMakeLists.txt.orig 2020-01-07 20:36:40.000000000 +0000
+++ icewm-1.6.4/src/CMakeLists.txt      2020-02-05 00:28:35.392235597 +0000
@@ -505,7 +505,7 @@
  target_compile_options(icewmhint${EXEEXT} PUBLIC ${CXXFLAGS_COMMON} 
${icewm_pc_flags})
  TARGET_LINK_LIBRARIES(icewmhint${EXEEXT} ${xext_LDFLAGS} ${x11_LDFLAGS} 
${nls_LIBS} ${EXTRA_LIBS})
-ADD_EXECUTABLE(icesh${EXEEXT} icesh.cc misc.cc mstring.cc ref.cc yarray.cc)
+ADD_EXECUTABLE(icesh${EXEEXT} icesh.cc misc.cc mstring.cc ref.cc yarray.cc 
ytimer.cc)
  target_compile_options(icesh${EXEEXT} PUBLIC ${CXXFLAGS_COMMON} 
${icewm_pc_flags})
  TARGET_LINK_LIBRARIES(icesh${EXEEXT} ${xext_LDFLAGS} ${xrandr_LDFLAGS} 
${xinerama_LDFLAGS} ${x11_LDFLAGS} ${nls_LIBS} ${EXTRA_LIBS})

It appears that the sed and patch do the same thing.  I'll update.

  -- Bruce


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

Reply via email to