Hi --

I noticed when trying to build Darcs with git support (just to play around), the
linker was reporting that it couldn't find the SHA1 functions (even though
./configure correctly detected them in -lcrypto).  After poking at your
Makefiles for a bit, I came up with the following patch.

I did notice the patch has the relatively-harmless side effect of including some
libraries twice.  On the other hand, modern linkers are smart enough to only
take what you need anyway, so I don't think it will be an issue.

Hope this helps.

-- Josh

Wed Jul 27 20:16:09 PDT 2005  Joshua J. Berry <[EMAIL PROTECTED]>
  * Include autoconf-detected libs in LDFLAGS
  Autoconf uses @LIBS@ -- not @LDFLAGS@ -- for libraries it detects (e.g. using
  AC_SEARCH_LIBS).

New patches:

[Include autoconf-detected libs in LDFLAGS
Joshua J. Berry <[EMAIL PROTECTED]>**20050728031609
 Autoconf uses @LIBS@ -- not @LDFLAGS@ -- for libraries it detects (e.g. using
 AC_SEARCH_LIBS).
] {
hunk ./autoconf.mk.in 18
-LDFLAGS		:= @LDFLAGS@
+LDFLAGS		:= @LDFLAGS@ @LIBS@
}

Context:

[new changelog entries.
David Roundy <[EMAIL PROTECTED]>**20050726123329] 
[clean up formatting in Depends.
David Roundy <[EMAIL PROTECTED]>**20050723130807] 
[changelog entry for fix to RT#208.
David Roundy <[EMAIL PROTECTED]>**20050722113803] 
[make make_changelog a bit more flexible in its parsing.
David Roundy <[EMAIL PROTECTED]>**20050722113701
 One can now have blank lines between the match: lines and the actual
 comments.
] 
[give better error message when dealing with a non-repository.
David Roundy <[EMAIL PROTECTED]>**20050722105908] 
[make make_changelog ignore boring files (emacs backups) in changelog.in/entries/.
David Roundy <[EMAIL PROTECTED]>**20050726121455] 
[fix bug in get_patches_beyond_tag that broke get --partial.
David Roundy <[EMAIL PROTECTED]>**20050723125507
 The bug was that we sometimes looked at patches that weren't strictly
 necesary.  This was because of the concat in get_patches_beyond_tag, which
 loses information about tag dependencies.  A clean implementation of a
 get_extra that accepts a true PatchSet would be a nicer fix (since it might
 fix other similar problems), but this fix is also correct and simple.
] 
[add changelog entry for get --partial fix.
David Roundy <[EMAIL PROTECTED]>**20050723130715] 
[scrunch up the tla/cvs tables a bit in the manual.
David Roundy <[EMAIL PROTECTED]>**20050724181011] 
[another alternative formatting for cvs/tla tables.
Erik Schnetter <[EMAIL PROTECTED]>**20050724134656] 
[alternative formatting for cvs/tla tables.
Erik Schnetter <[EMAIL PROTECTED]>**20050724113905] 
[make add/remove --list-options not output preceding ./
David Roundy <[EMAIL PROTECTED]>**20050723134758
 We were treating the repository root differently from subdirectories
 because the file paths didn't need to get "fixed".  Addresses bug #158.
] 
[fix unit test that prompts for input
Will <[EMAIL PROTECTED]>**20050722181028] 
[Make DarcsRepo.add_to_inventory take a list.
Juliusz Chroboczek <[EMAIL PROTECTED]>**20050720174029
 This avoids opening the inventory multiple times.  Thanks to Ian for the hint.
] 
[Use mapM_ instead of the comprehensible alternative.
Juliusz Chroboczek <[EMAIL PROTECTED]>**20050720164258
 Mentioning mapM_ always impresses people at dinner parties.  Thanks to
 Ian for the hint.
] 
[Move iterateGitTree out of the IO monad.
Juliusz Chroboczek <[EMAIL PROTECTED]>**20050720162841
 We're reading immutable on-disk data, it's safe to do it unsafely.
] 
[Clean up usage of interleaveIO in Git.
Juliusz Chroboczek <[EMAIL PROTECTED]>**20050720162251] 
[fix write_problem to show all problems.
David Roundy <[EMAIL PROTECTED]>**20050717110628] 
[don't import head and tail, which are in the prelude.
David Roundy <[EMAIL PROTECTED]>**20050716143547] 
[Rename bound variable in fromJust macro.
Juliusz Chroboczek <[EMAIL PROTECTED]>**20050716221705
 Avoids ``shadows existing variable'' warnings which for some reason are
 errors.
 
 Could we please use Lisp macros instead?
] 
[TAG 2005-07-18
Ian Lynagh <[EMAIL PROTECTED]>**20050718193534] 
[put configure.ac back in the public domain.
David Roundy <[EMAIL PROTECTED]>**20050720115702] 
[fix error in name of --reorder-patches flag.
David Roundy <[EMAIL PROTECTED]>**20050722110752] 
[Push and pull can now show the detailed diffs of patches
Jim Radford <[EMAIL PROTECTED]>**20050717042645
 The same distinction is now made between --summary and --verbose
 as changes makes.
] 
[advance DARCS_VERSION to 1.0.4pre2.
David Roundy <[EMAIL PROTECTED]>**20050720115536
 In the new tradition of changing the version after a release rather than
 before a release (although when the release type changes to rc or actual
 release it'll have to be done before the release).
] 
[drop $srcdir use; build-directories aren't supported anyway
Peter Simons <[EMAIL PROTECTED]>**20050719140044] 
[clean generated manual files at realclean
Peter Simons <[EMAIL PROTECTED]>**20050719135935] 
[cosmetic changes
Peter Simons <[EMAIL PROTECTED]>**20050719135834] 
[move comment to the right place
Peter Simons <[EMAIL PROTECTED]>**20050719135818] 
[let config.status generate config.command
Peter Simons <[EMAIL PROTECTED]>**20050719135733] 
[make use of autoconf 2.5x's AC_INIT macro
Peter Simons <[EMAIL PROTECTED]>**20050719135611] 
[use ./config.status to re-configure build after autoconf changes
Peter Simons <[EMAIL PROTECTED]>**20050719135435] 
[update distclean and realclean targets
Peter Simons <[EMAIL PROTECTED]>**20050719135415] 
[canonize [EMAIL PROTECTED]
Peter Simons <[EMAIL PROTECTED]>**20050719134834] 
[cosmetic change
Peter Simons <[EMAIL PROTECTED]>**20050719134816] 
[update test suite to work with Peter's makefile changes.
David Roundy <[EMAIL PROTECTED]>**20050721102319] 
[TAG 1.0.4pre1
David Roundy <[EMAIL PROTECTED]>**20050718112234] 
Patch bundle hash:
1abd9ef0e257cdd18e25aac9ab972a237df6d5e5
_______________________________________________
darcs-devel mailing list
[email protected]
http://www.abridgegame.org/cgi-bin/mailman/listinfo/darcs-devel

Reply via email to