[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

Re: RFS: falcon package (ITP:Bug#460591); source package



Cyril Brulebois wrote:
On 14/01/2008, Giancarlo Niccolai wrote:
It is a sightly modified version of the package I am preparing for
Ubuntu, and it should generate correctly the libraries, the binaries,
the -dev and -dbg packages related to the basic contents of the Falcon
Language.

Some quick remarks (this is not a complete review):
 - You need to target unstable rathen than sid.
 - Your Standards-Version is outdated.
 - You don't need the g++ B-D.
Ok, I will fix this

 - What is debian/copyfiles? Don't dh_install do the job already?
 - I think the ldconfig call will be added automatically by debhelper,
   to check. Same thing for the symlink.
 - No need to chown/chmod debian/tmp, dh_fixperms will do.
 - Looks to me that debian/rules could be really simplified if you were
   using dh_* commands.
Yes, they could; but the build process is quite customized. Source comes from three sub-projects that are kept separate for administrative reasons, but are integrated at operating level. I read in the debhelper and related manual that it may have some trouble in detecting modules which need particular stripping rules, and a very important part of the base library (which is supposed to support application embedding scripts) relies on loadable binary modules, as the RTL.

If that has to be done manually, then using some automated tools and not other may mess up things; if not now, they may be in future if new dependencies are drawn between dh_* tools.

I may consider using dh_fixperms, though; it seems independent enough.

And anyhow, I may use anything if it simplifies the process and works, but I'd need assistance, as using such automated (and documented-through-examples) tools can be problematic when having custom builds and not a clear picture on how they infer things.

I think there's something wrong with the watch file. Also, I need a
spin for what concerns the checksum files. All the rest should be
pretty complete.

You might want to use the download page[1] although it doesn't contain
any link to .7 currently.

 1. http://www.falconpl.org/?page_id=downloads
Oh... so the watch file can refer even an http page linking the packages? -- where can I find some example?
However, shouldn't it work also on i.e. the http tree at

http://falconpl.org/downloads/

Sorry, I am really a bit confused about the watch file, but I need a bit of direction here.

Thanks,
Giancarlo Niccolai.


Reply to: