New ebuild for CAMFR, a full vectorial Maxwell solver based on eigenmode expansion. http://camfr.sourceforge.net/ Reproducible: Always Steps to Reproduce: 1. 2. 3.
Created attachment 22708 [details] camfr ebuild
Created attachment 22709 [details] machine_cfg.py
Created attachment 22710 [details] SConstruct
Ok, I am coaching him, I'll take that. Geirge
Looking clean conceptually, but the same thing with dependencies. It is helpfull to run "repoman scan" on the package before you submit it ;). Also, I see it depends on updated version of blitz, which is #25731, so this bug should have a corresponding dependency as well. Added it now.. Another thing, this one also uses blas/lapack, so it'll have to wait untill the reorganization, to which I'll try to get ASAP. But I would like to get done with your bugs at least conceptually now. George
Oops, forgot to adde dependency :(. Also, Peter, could you please add ChangeLog here as well? (no need to add metadada ehere if you do it for #36581, or add it here and skip there - this one is quite trivial). George
Created attachment 23922 [details] Changelog Added Changelog
Created attachment 24273 [details] ebuild Check for Tk support
Created attachment 24274 [details] ebuild uploaded wrong version
Hi Peter. Thanks for update, however I spotted a problem with the following block in src_unpack: python -c "import ImageTk" 2>/dev/null || einfo "Python and Imaging don't have Tk support enabled." || einfo "Set the tcltk USE flag and reinstall Python and Imaging before continuing." || die "Configuration error" 1. The check should really be moved to pkg_setup (pkg_* functions have access to the outside of the sandbox, so this is usefull for some setup stuff). While it does not really matter when the package is built from source, it makes a great difference for prebuilt package (and we need to support both) - src_unpack is not run in that case! 2. In present form only the first echoi after || gets executed, you need to group the echos and die together in a different way, for example using the genuine if [ ... ]; then ... fi (plus it will be more readable). George
Created attachment 24342 [details] ebuild Incorporated your feedback.
Created attachment 24487 [details] ebuild Adds dependency on ps2pdf (in app-text/xdpf)
Ok, looking good now, but I see it also depends on lapack. So please do not committ it (or other blas/lapack ebuilds, unless they are updates) when you get cvs access untill the big blas reorganization is complete (#30453, btw, your opinion on the decided solution is wellcome ;)). George
Created attachment 24806 [details] machine_cfg.py Update machine_cfg.py: add whitespace when concatenating compiler options
Added to unstable tree.
