Skip to content
This repository has been archived by the owner on Jul 4, 2023. It is now read-only.

New formula for OpenJade #14026

Closed
wants to merge 3 commits into from
Closed

New formula for OpenJade #14026

wants to merge 3 commits into from

Conversation

theirix
Copy link
Contributor

@theirix theirix commented Aug 7, 2012

Formula for OpenJade (http://openjade.sourceforge.net), DSSSL
SGML processor. Port is derived from MacPorts, uses all
provided patches and one more patch about missing .la file.
Port requires LLVM compiler, open-sp built with LLVM and
recently added xmlcatmgr.

@michaelpq
Copy link
Contributor

+1, this is very helpful when working on SGML documentation, particularly PostgreSQL.

@adamv
Copy link
Contributor

adamv commented Sep 13, 2012

A little uncomfortable about adding something that requires LLVM, as Clang is Apple's compiler going forward.

@theirix
Copy link
Contributor Author

theirix commented Sep 16, 2012

Unfortunately my previous efforts to update open-jade to compile with clang failed - very old code. Anyway I'll try to do this again.

@adamv
Copy link
Contributor

adamv commented Sep 25, 2012

Please squash to a single commit for review, thanks.

@adamv
Copy link
Contributor

adamv commented Nov 1, 2012

Can some other maintainers review this? I'm uncomfortable with the rescue blocks here.

theirix and others added 3 commits November 6, 2012 14:52
Formula for OpenJade (http://openjade.sourceforge.net), DSSSL
SGML processor. Port is derived from MacPorts, uses all
provided patches and one more patch about missing .la file.
Clarified comments.
Fixed a bug when patch order was unspecified for ruby 1.8.
Added configure flags for catalog and data pathes.
@dimitri
Copy link

dimitri commented Nov 30, 2012

That formula fails here:

==> Installing open-jade
==> Downloading http://downloads.sourceforge.net/project/openjade/openjade/1.3.2/openjade-1.3.2.tar.gz
## 100.0%
==> Downloading patches
## 100.0%
## 100.0%
## 100.0%
## 100.0%
## 100.0%
## 100.0%
## 100.0%
## 100.0%
## 100.0%
==> Patching
patching file Makefile.lib.in
patching file Makefile.prog.in
patching file config/ltmain.sh
patching file configure
Hunk #1 succeeded at 3063 with fuzz 1 (offset 186 lines).
Hunk #2 succeeded at 7907 (offset 72 lines).
Hunk #3 succeeded at 9365 (offset 79 lines).
patching file spgrove/GroveApp.h
patching file spgrove/GroveBuilder.cxx
patching file grove/Node.h
/usr/bin/patch: **** Only garbage was found in the patch input.
Error: Failure while executing: /usr/bin/patch -f -p0 -i 007-homebrew.diff

@adamv
Copy link
Contributor

adamv commented Dec 15, 2012

Still needs to be squashed to a single commit

@adamv
Copy link
Contributor

adamv commented Jan 15, 2013

Did not build for me:

==> make install
make install
/bin/sh /private/tmp/open-jade-i2D2/openjade-1.3.2/config/mkinstalldirs /Users/adamv/homebrew/Cellar/open-jade/1.3.2/lib
mkdir /Users/adamv/homebrew/Cellar/open-jade/1.3.2/lib
/bin/sh /private/tmp/open-jade-i2D2/openjade-1.3.2/libtool --mode=install config/install-sh -c  libogrove.la /Users/adamv/homebrew/Cellar/open-jade/1.3.2/lib
config/install-sh -c .libs/libogrove.0.0.1.dylib /Users/adamv/homebrew/Cellar/open-jade/1.3.2/lib/libogrove.0.0.1.dylib
/private/tmp/open-jade-i2D2/openjade-1.3.2/libtool[4440]: eval: line 1: config/install-sh: not found
make[2]: *** [install] Error 127
make[1]: *** [grove] Error 2
make: *** [install] Error 2

@@ -23,7 +23,7 @@ def latest_version
when 10.6 then "3.2.6"
else
if MacOS.version >= 10.7
"4.5.1"
"4.5.2"
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This shouldn't be here.

@MikeMcQuaid
Copy link
Member

Closing until this builds

@MikeMcQuaid MikeMcQuaid closed this Feb 1, 2013
@theirix theirix deleted the open-jade branch November 4, 2013 21:27
@theirix theirix restored the open-jade branch November 4, 2013 21:27
@theirix theirix mentioned this pull request Nov 4, 2013
@Homebrew Homebrew 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.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants