appleseed icon indicating copy to clipboard operation
appleseed copied to clipboard

Remove all occurrences of `foundation::each<>` and `foundation::const_each<>`

Open dictoon opened this issue 5 years ago • 15 comments

Replace them by range-based for loops.

dictoon avatar Sep 21 '19 23:09 dictoon

You mentioned why somewhere but I forgot where 🤦‍♂ Then my question is why ?

oktomus avatar Sep 22 '19 11:09 oktomus

Since C++11 introduced range-based for loops, we no longer need our custom mechanism anymore: it doesn't bring anything over range-based for loops. The only case where a similar custom mechanism could be useful is for reverse iteration.

dictoon avatar Sep 22 '19 21:09 dictoon

If no one is working on this I would like to take it

hbina avatar Sep 28 '19 03:09 hbina

Go ahead @hbina :)

oktomus avatar Sep 28 '19 07:09 oktomus

@oktomus I have been trying to extract the tar file here https://github.com/appleseedhq/linux-deps/releases/download/v2.1/appleseed-deps-shared-2.1.tgz but for whatever reason I have insufficient permission. Do I really need to sudo?

Edit : I managed to extract it but I am having troubles compiling with cmake...It's a strange process to say the least.

Target architecture: x86_64
-- Using gcc 7 as the compiler
-- Found Boost: /home/hbina085/Documents/prebuilt-linux-deps/include/boost_1_61_0 (found suitable version "1.61.0", minimum required is "1.61") found components: atomic chrono date_time filesystem regex system thread wave missing components: serialization
-- Found LZ4: /home/hbina085/Documents/prebuilt-linux-deps/lib/liblz4.so
-- Found Boost: /home/hbina085/Documents/prebuilt-linux-deps/include/boost_1_61_0 (found suitable version "1.61.0", minimum required is "1.61") found components: python 
-- Configuring done
CMake Warning at src/appleseed/CMakeLists.txt:2276 (add_library):
  Cannot generate a safe runtime search path for target appleseed because
  files in some directories may conflict with libraries in implicit
  directories:

    runtime library [libz.so] in /usr/lib/x86_64-linux-gnu may be hidden by files in:
      /home/hbina085/Documents/prebuilt-linux-deps/lib

  Some of these libraries may not be found correctly.


CMake Warning at src/appleseed.shared/CMakeLists.txt:67 (add_library):
  Cannot generate a safe runtime search path for target appleseed.shared
  because files in some directories may conflict with libraries in implicit
  directories:

    runtime library [libz.so] in /usr/lib/x86_64-linux-gnu may be hidden by files in:
      /home/hbina085/Documents/prebuilt-linux-deps/lib

  Some of these libraries may not be found correctly.


CMake Warning at src/appleseed.cli/CMakeLists.txt:54 (add_executable):
  Cannot generate a safe runtime search path for target appleseed.cli because
  files in some directories may conflict with libraries in implicit
  directories:

    runtime library [libz.so] in /usr/lib/x86_64-linux-gnu may be hidden by files in:
      /home/hbina085/Documents/prebuilt-linux-deps/lib

  Some of these libraries may not be found correctly.


CMake Warning at src/appleseed.python/CMakeLists.txt:148 (add_library):
  Cannot generate a safe runtime search path for target appleseed.python
  because files in some directories may conflict with libraries in implicit
  directories:

    runtime library [libz.so] in /usr/lib/x86_64-linux-gnu may be hidden by files in:
      /home/hbina085/Documents/prebuilt-linux-deps/lib

  Some of these libraries may not be found correctly.


CMake Warning at src/appleseed.studio/CMakeLists.txt:476 (add_executable):
  Cannot generate a safe runtime search path for target appleseed.studio
  because files in some directories may conflict with libraries in implicit
  directories:

    runtime library [libz.so] in /usr/lib/x86_64-linux-gnu may be hidden by files in:
      /home/hbina085/Documents/prebuilt-linux-deps/lib

  Some of these libraries may not be found correctly.


CMake Warning at src/tools/animatecamera/CMakeLists.txt:54 (add_executable):
  Cannot generate a safe runtime search path for target animatecamera because
  files in some directories may conflict with libraries in implicit
  directories:

    runtime library [libz.so] in /usr/lib/x86_64-linux-gnu may be hidden by files in:
      /home/hbina085/Documents/prebuilt-linux-deps/lib

  Some of these libraries may not be found correctly.


CMake Warning at src/tools/convertmeshfile/CMakeLists.txt:52 (add_executable):
  Cannot generate a safe runtime search path for target convertmeshfile
  because files in some directories may conflict with libraries in implicit
  directories:

    runtime library [libz.so] in /usr/lib/x86_64-linux-gnu may be hidden by files in:
      /home/hbina085/Documents/prebuilt-linux-deps/lib

  Some of these libraries may not be found correctly.


CMake Warning at src/tools/denoiser/CMakeLists.txt:49 (add_executable):
  Cannot generate a safe runtime search path for target denoiser because
  files in some directories may conflict with libraries in implicit
  directories:

    runtime library [libz.so] in /usr/lib/x86_64-linux-gnu may be hidden by files in:
      /home/hbina085/Documents/prebuilt-linux-deps/lib

  Some of these libraries may not be found correctly.


CMake Warning at src/tools/dumpmetadata/CMakeLists.txt:51 (add_executable):
  Cannot generate a safe runtime search path for target dumpmetadata because
  files in some directories may conflict with libraries in implicit
  directories:

    runtime library [libz.so] in /usr/lib/x86_64-linux-gnu may be hidden by files in:
      /home/hbina085/Documents/prebuilt-linux-deps/lib

  Some of these libraries may not be found correctly.


CMake Warning at src/tools/makefluffy/CMakeLists.txt:51 (add_executable):
  Cannot generate a safe runtime search path for target makefluffy because
  files in some directories may conflict with libraries in implicit
  directories:

    runtime library [libz.so] in /usr/lib/x86_64-linux-gnu may be hidden by files in:
      /home/hbina085/Documents/prebuilt-linux-deps/lib

  Some of these libraries may not be found correctly.


CMake Warning at src/tools/projecttool/CMakeLists.txt:52 (add_executable):
  Cannot generate a safe runtime search path for target projecttool because
  files in some directories may conflict with libraries in implicit
  directories:

    runtime library [libz.so] in /usr/lib/x86_64-linux-gnu may be hidden by files in:
      /home/hbina085/Documents/prebuilt-linux-deps/lib

  Some of these libraries may not be found correctly.


-- Generating done
-- Build files have been written to: /home/hbina085/git/appleseed/build

Edit : Fixed that...now I have a different problem xd

I don't understand the part about setting up the local Python environment at all...

Edit: Its segfaulting...

hbina avatar Sep 29 '19 02:09 hbina

@hbina For any problem you encounter with compilation, you should ask for help in #compiling on our discord https://discord.gg/dNCE5J8

oktomus avatar Sep 29 '19 16:09 oktomus

hi is this issue still open? I would like to work on it

darthgera123 avatar Dec 08 '19 23:12 darthgera123

Yes it is still open.

dictoon avatar Dec 10 '19 11:12 dictoon

I would like to work on this. Is it still open?

prat31 avatar Feb 21 '20 12:02 prat31

It is.

dictoon avatar Feb 22 '20 14:02 dictoon

hey @dictoon! Is this issue still open?

AyushRaj8271 avatar May 18 '21 05:05 AyushRaj8271

@AyushRaj8271 It's possible that the last PR #2869 doesn't replace all occurences. You can check in the code

oktomus avatar May 18 '21 06:05 oktomus

Hello, is this issue still open ? If yes I would like to work on it. Can you please assign me this issue.

ganesh1828 avatar Mar 30 '23 05:03 ganesh1828

Hi @ganesh1828 ,you can work on it directly and submit a PR.

oktomus avatar Mar 30 '23 10:03 oktomus

Hi @oktomus / @dictoon I think this issue is solved in #2869 PR. You can close this issue.

ganesh1828 avatar Oct 16 '23 06:10 ganesh1828