Skip to content

HTTPS clone URL

Subversion checkout URL

You can clone with
or
.
Download ZIP

Loading…

duti repo now hosted on github, update URLs #15092

Closed
wants to merge 2 commits into from

2 participants

@moretension

I'm the maintainer of the duti project. I've moved the project repo from SF.net to github. Thought you'd appreciate a patch for duti.rb with the new URLs.

@adamv

What's the reasoning here? Is it no longer pre-autoconfed?

Andrew Mortensen Use 1.5.1 release imported from SF.net instead of 1.5.1 tag tarball.
Eliminate autoconf dependency.
164148c
@moretension

I generally don't version control the configure script (autoconf before release instead), so the 1.5.1 tag doesn't have a configure script. I may change that for future updates, but for now I've added a github download URL that's a tarball of 1.5.1 with the configure script. I'll send another pull request.

@adamv
Owner

This now results in a build failure on Snow Leopard; PPC arch isn't compatible with the other flags we set, and shouldn't be a compiled architecture.

$ brew install -v duti
==> Downloading https://github.com/downloads/fitterhappier/duti/duti-1.5.1.tar.gz
/usr/bin/curl -qf#LA Homebrew 0.9.3 (Ruby 1.8.7-249; Mac OS X 10.6.8) https://github.com/downloads/fitterhappier/duti/duti-1.5.1.tar.gz -o /usr/local/homebrew-cache/duti-1.5.1.tar.gz
######################################################################## 100.0%
/usr/bin/tar xf /usr/local/homebrew-cache/duti-1.5.1.tar.gz
==> ./configure --prefix=/Users/adamv/homebrew/Cellar/duti/1.5.1
./configure --prefix=/Users/adamv/homebrew/Cellar/duti/1.5.1
checking for gawk... gawk
checking for gcc... /usr/bin/gcc-4.2
checking for C compiler default output file name... a.out
checking whether the C compiler works... yes
checking whether we are cross compiling... no
checking for suffix of executables... 
checking for suffix of object files... o
checking whether we are using the GNU C compiler... yes
checking whether /usr/bin/gcc-4.2 accepts -g... yes
checking for /usr/bin/gcc-4.2 option to accept ISO C89... none needed
checking for a BSD-compatible install... /usr/bin/install -c
checking build system type... i386-apple-darwin10.8.0
checking host system type... i386-apple-darwin10.8.0
checking target system type... i386-apple-darwin10.8.0
checking which SDK to use... /Developer/SDKs/MacOSX10.6.sdk
checking Mac OS X deployment target... 10.6
checking for strlcpy... yes
configure: creating ./config.status
config.status: creating Makefile
config.status: creating version.c
==> make install
make install 
/usr/bin/gcc-4.2 -isysroot /Developer/SDKs/MacOSX10.6.sdk -arch i386 -arch ppc -mmacosx-version-min=10.6 -Wall -Wmissing-prototypes -Os -w -pipe -march=core2 -mmacosx-version-min=10.6 -c ./version.c
/usr/bin/gcc-4.2 -isysroot /Developer/SDKs/MacOSX10.6.sdk -arch i386 -arch ppc -mmacosx-version-min=10.6 -Wall -Wmissing-prototypes -Os -w -pipe -march=core2 -mmacosx-version-min=10.6 -c ./util.c
cc1: error: unrecognized command line option "-march=core2"
lipo: can't open input file: /var/folders/-R/-RTga5RvF-8UNERefp3d5++++TI/-Tmp-//ccWeSaJz.out (No such file or directory)
make: *** [version.o] Error 1
make: *** Waiting for unfinished jobs....
cc1: error: unrecognized command line option "-march=core2"
lipo: can't open input file: /var/folders/-R/-RTga5RvF-8UNERefp3d5++++TI/-Tmp-//ccVI64UT.out (No such file or directory)
make: *** [util.o] Error 1
Error: Failure while executing: make install 
[~/homebrew] (duti)
$ 
@moretension

The problematic -march=... flag seems to be added by homebrew, in Library/Homebrew/extend/ENV.rb.

@adamv
Owner

I know where the flag is added; we need to patch duti to not try to build ppc/i386 on Snow Leopard, and build 32-bit or 64-bit according to the normal rules that Homebrew follows.

@adamv
Owner

Line 2898 in ./configure is hard-coding arches based on OS X version; need to be able to pass in just 32-bit or 64-bit as appropriate.

@adamv adamv closed this pull request from a commit
Andrew Mortensen duti 1.5.1
Closes #15092.

Signed-off-by: Adam Vandenberg <flangy@gmail.com>
73741aa
@adamv adamv closed this in 73741aa
@asharpe asharpe referenced this pull request from a commit
Andrew Mortensen duti 1.5.1
Closes #15092.

Signed-off-by: Adam Vandenberg <flangy@gmail.com>
f119d7f
@norioxkimura norioxkimura referenced this pull request from a commit in norioxkimura/homebrew
Andrew Mortensen duti 1.5.1
Closes #15092.

Signed-off-by: Adam Vandenberg <flangy@gmail.com>
1334004
@cooljeanius cooljeanius referenced this pull request from a commit
Commit has since been removed from the repository and is no longer available.
@rajeeja rajeeja referenced this pull request from a commit
Commit has since been removed from the repository and is no longer available.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Commits on Sep 25, 2012
  1. Update with github URLs, add autoconf dependency.

    Andrew Mortensen authored
Commits on Sep 26, 2012
  1. Use 1.5.1 release imported from SF.net instead of 1.5.1 tag tarball.

    Andrew Mortensen authored
    Eliminate autoconf dependency.
This page is out of date. Refresh to see the latest.
Showing with 4 additions and 2 deletions.
  1. +4 −2 Library/Formula/duti.rb
View
6 Library/Formula/duti.rb
@@ -1,9 +1,11 @@
require 'formula'
class Duti < Formula
- homepage 'http://duti.sourceforge.net/'
- url 'http://downloads.sourceforge.net/project/duti/duti/duti-1.5.1/duti-1.5.1.tar.gz'
+ homepage 'http://duti.org/'
+ url 'https://github.com/downloads/fitterhappier/duti/duti-1.5.1.tar.gz'
sha1 'ac199f936180a3ac62100ae9a31e107a45330557'
+ head 'https://github.com/fitterhappier/duti.git'
+
def install
system "./configure", "--prefix=#{prefix}"
Something went wrong with that request. Please try again.