On Sun, Dec 30, 2018 at 11:18:32PM +0200, Hannu Vuolasaho wrote:
> svn has fixed those too... and man page PREFIX expansion is on TODO.
> So next release should be patchless.
Great!
> There are also --symbols and --profile configure options. The
> debugging option is
> mainly used for portability check with std=c89.
Hm, CFLAGS is currently ignored completely and I missed this previously.
Passing it through CONFIGURE_ARGS will propagate DEBUG for us, so let's
use that.
However, I need to remove the `--debug' flag, otherwise it overwrites
stuff again. Since the configure script is not a script but a binary
being compiled right before execution without proper usage, I'm
reluctant to look any further into this.
> This is harder thing than I originally thought. SUBST_CMD doesn't have effect
> unless braces aren't patched to man page. The man page is installed with cp.
Correct, for using SUBST_CMD sources are usually patched. `cp' is fine
here as the manual ends up with proper permissions.
> There could be post-install target which would delete the installed man page and
> use and install it again with SUBST_MAN which isn't elegant solution.
> The PREFIX expansion wil happen by xschem and install method will change in
> next release, the search for most beautiful installation method is
> rather useless
> as long it is good enough.
Ok, thanks for your work with upstream again.
> First of all I don't know how to disable all the unsupported simulators and
> waveform viewers without causing side-effects.
>
> This release supports Icarus verilog, GHDL and GTKWave from open source
> tool chains and those are for digital only. For analog side there are only
> commercial spices available.
>
> The reason for getting xschem to ports is its capability export tEDAx netlists
> which pcb-rnd can use and I'm working on that. I have few show stoppers there.
>
> Next release of xschem will also support ngspice and maybe the waveform viewer
> problem can be also solved.
Sounds like stuff we can work on after this port got in.
> So I'll try get the upstream to give some configuation option to hide the
> unsupported simulators and viewers.
Cool.
> Attached today's work.
Looks much better. I've made a few minor tweaks:
* zap trailing tab in WANTLIB line, sort libs
* add some newlines and slightly reorder MOD* variables for readability
* pass CFLAGS, drop `--debug' in CONFIGURE_ARGS
* break LIB_DEPENDS into one port per line
* quote variables
* moved targets (pre-configure) to the end as usual
* zap useless NO_TEST comment
What do you think?
Feedback? OK to import?
No comments:
Post a Comment