http://analog.cx/analog-6.0 #18104

Closed
wants to merge 2 commits into
from

2 participants

@brendanjerwin

Shameless rip of a seemingly abandoned pull request:
#15016

It works for me, hopefully it just needed some branch and git tidy...

@adamv

This and the next line are bin.install 'analog'

@adamv

Rewrite these few lines to use install too.

@brendanjerwin brendanjerwin Reworked to use the `.install` magic
Also adjusted the `-DLANGDIR` to not require the `analog` subdir in
`share`. Seemed wasteful.
ec3df67
@brendanjerwin

@adamv There ya go. It got a lot tighter. That's a nifty DSL. Where can I look to learn more about it?

@adamv

We're under-documented at the moment, unfortunately.

@adamv

We use a special form, depends_on :libpng for png support, as it is sometimes provided by the system X11.

@adamv

Should be (share/'analog').install "examples", "how-to", "images", "lang"

@adamv

I'll make the remaining edits

@adamv

Er, we can't install "analog.cfg" into the bin folder, as it is not an executable file.

@adamv

Running the test gives me:

$ brew test -v analog
Testing analog
==> /Users/adamv/homebrew/Cellar/analog/6.0/bin/analog > /dev/null
/Users/adamv/homebrew/Cellar/analog/6.0/bin/analog > /dev/null 
/Users/adamv/homebrew/Cellar/analog/6.0/bin/analog: analog version 6.0/Unix
/Users/adamv/homebrew/Cellar/analog/6.0/bin/analog: Warning D: Turning all pie
  charts off because OUTFILE is stdout and CHARTDIR is unset
  (For help on all errors and warnings, see docs/errors.html)
/Users/adamv/homebrew/Cellar/analog/6.0/bin/analog: Warning F: Failed to open
  logfile /Users/adamv/homebrew/Cellar/analog/6.0/bin/logfile.log: ignoring it
/Users/adamv/homebrew/Cellar/analog/6.0/bin/analog: Warning R: Turning off
  empty time reports
/Users/adamv/homebrew/Cellar/analog/6.0/bin/analog: Warning R: Turning off
  empty Request Report
/Users/adamv/homebrew/Cellar/analog/6.0/bin/analog: Warning R: Turning off
  empty File Type Report
/Users/adamv/homebrew/Cellar/analog/6.0/bin/analog: Warning R: Turning off
  empty Directory Report
/Users/adamv/homebrew/Cellar/analog/6.0/bin/analog: Warning R: Turning off
  empty Domain Report
/Users/adamv/homebrew/Cellar/analog/6.0/bin/analog: Warning R: Turning off
  empty Organisation Report
/Users/adamv/homebrew/Cellar/analog/6.0/bin/analog: Warning R: Turning off
  empty Search Word Report
/Users/adamv/homebrew/Cellar/analog/6.0/bin/analog: Warning R: Turning off
  empty Operating System Report
/Users/adamv/homebrew/Cellar/analog/6.0/bin/analog: Warning R: Turning off
  empty File Size Report
/Users/adamv/homebrew/Cellar/analog/6.0/bin/analog: Warning R: Turning off
  empty Status Code Report

We can't be saving log files in the same folder as the exe either.

@brendanjerwin

analog wants a default .cfg file next to the executable. If the .cfg shouldn't go in bin How is that addressed?

The remaining items then:

  • move the cfg out of bin
  • move logging location out of bin
  • (share/'analog').install
@adamv

Looks like the log location can be changed in the .cfg file.
The bin and .cfg can be installed side-by-side into libexec, and then symlinked into bin it seems.

But the makefile in src also needs to be edited to not hard-code GCC and certain make flags.

Passing on this until an updated pull request comes in.

@adamv adamv closed this May 15, 2013
@xu-cheng xu-cheng locked and limited conversation to collaborators Feb 16, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.