public inbox for cygwin-apps@cygwin.com
 help / color / mirror / Atom feed
From: Charles Wilson <cygwin@cwilson.fastmail.fm>
To: Mailing List: CygWin-Apps <cygwin-apps@cygwin.com>
Subject: Re: [ITP] astrometry.net-0.38-1
Date: Fri, 04 Nov 2011 21:13:00 -0000	[thread overview]
Message-ID: <4EB45567.5030500@cwilson.fastmail.fm> (raw)
In-Reply-To: <4EB3865A.2060400@cwilson.fastmail.fm>

On 11/4/2011 2:29 AM, Charles Wilson wrote:
> Your build still links against libbackend.a, rather than cygbackend.dll.
>
> I'm trying to massage your -src package to DTRT. Stay tuned.

I've posted a revised version of your package here:

http://cygwin.cwilson.fastmail.fm/astrometry.net-0.38-2-src.tar.bz2
http://cygwin.cwilson.fastmail.fm/astrometry.net-0.38-2.tar.bz2

Inside the -src tarball is the *original* upstream sources 
(astrometry.net-0.38.tar.bz2), a few patches, and a .cygport script. to 
rebuild, unpack the -src and do:

	$ cygport astrometry.net-0.38-2.cygport all

You should probably do that, to ensure that the build procedure works on 
your machine. Also, to test the resuts; I have no idea how to use this 
stuff.

Finally: the backend.exe program is linked to cygbackend.dll, which are 
both in /usr/lib/astrometry/bin/.  All the python stuff, including three 
extension DLLs, are in /usr/lib/astrometry/python/*/

I modified the build procedure for cygbackend.dll -- it now generates an 
import lib, and it also (re)exports all of the symbols from the other 
(sub)libraries.  That means when linking backend.exe, you don't need to 
list those other dependencies, because cygbackend.dll will satisfy them 
all. (*)

Provided you can rebuild this package on your machine, AND that it 
actually works, consider it GTG.

(*) I did not do this, but because cygbackend now exports all the 
symbols from (e.g.) libanfiles.a, libutils.a, libkd.a, etc, you COULD 
modify the link commands for almost all of the /other/ .exe's in the 
blind/ directory, which currently link against (many) of those static 
libraries, to instead link solely against cygbackend.dll (actually, 
libbackend.dll.a). That would make the entire package MUCH smaller.  But 
it's a lot of work.

--
Chuck

  parent reply	other threads:[~2011-11-04 21:13 UTC|newest]

Thread overview: 42+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-10-04  8:02 Jussi Kantola
2011-10-04  9:29 ` Corinna Vinschen
2011-10-05  8:43 ` Peter Li
2011-10-05 10:04   ` Jussi Kantola
2011-10-05 16:02 ` Charles Wilson
2011-10-11 10:19   ` Marco Atzeri
2011-10-18 11:21     ` Jussi Kantola
2011-10-18 11:49       ` Chris Sutcliffe
2011-10-18 21:12         ` Ken Brown
2011-10-20  8:19           ` Jussi Kantola
2011-10-30 21:10             ` Jussi Kantola
2011-11-03 12:03               ` Corinna Vinschen
2011-11-03 12:55                 ` Charles Wilson
2011-11-03 13:03                   ` Corinna Vinschen
2011-11-04  0:52                 ` Jussi Kantola
2011-10-05 16:19 ` Andrew Schulman
2011-10-06 13:06   ` Jussi Kantola
2011-10-07 14:20     ` Marco Atzeri
2011-10-07 16:19       ` Jussi Kantola
2011-10-07 16:20         ` Jussi Kantola
2011-10-10 18:54           ` Jussi Kantola
2011-10-10 23:14             ` Christopher Faylor
2011-11-04  6:30         ` Charles Wilson
2011-11-04  9:02           ` Jussi Kantola
2011-11-04 15:12           ` Yaakov (Cygwin/X)
2011-11-04 16:08             ` Charles Wilson
2011-11-04 21:13           ` Charles Wilson [this message]
2011-11-07 13:18             ` Jussi Kantola
2011-11-07 14:46               ` Charles Wilson
2011-11-07 16:13                 ` Jussi Kantola
2011-11-07 16:18                 ` Christopher Faylor
2011-11-08  4:50                   ` Charles Wilson
2011-11-08  7:19                     ` Christopher Faylor
2011-11-08 12:44                       ` Ken Brown
2011-11-09  2:43                         ` Chris Sutcliffe
2011-11-09 10:00                           ` Jussi Kantola
2011-11-09 23:24                             ` Charles Wilson
2011-11-11 10:46                               ` Jussi Kantola
2011-11-11 10:53                                 ` Jussi Kantola
2011-11-13 17:35                                   ` Jussi Kantola
2011-11-08 18:01                       ` Marco Atzeri
2011-11-08 19:55                         ` Christopher Faylor

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=4EB45567.5030500@cwilson.fastmail.fm \
    --to=cygwin@cwilson.fastmail.fm \
    --cc=cygwin-apps@cygwin.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).