Ticket #626 (closed task: fixed)

Opened 3 years ago

Last modified 17 months ago

Include raster_comments.sql in tar ball

Reported by: robe Owned by: strk
Priority: medium Milestone: PostGIS 2.0.0
Component: build/upgrade/install Version: trunk
Keywords: Cc:

Description

I added raster_comments as a build target when

make comments

is run in

doc\Makefile.in

I was hoping that was all I would need to do to have it included as part of the tar ball build. I don't see it included in the tar ball I downloaded, so guess there must be some other place this is set?

Change History

Changed 17 months ago by strk

  • owner changed from pramsey to strk
  • component changed from postgis to build/upgrade/install

Changed 17 months ago by strk

  • status changed from new to assigned

@robe: where do you download the tarball from ? @pramsey: is make_dist.sh still the script used to generate tarballs ?

Changed 17 months ago by strk

@robe: no "*_comments.sql" file is included in distribution as they are generated at build time. Are you suggesting we should change this to always include the generated comments file ? Note that we aren't distributing the enabling .sql files (also generated).

Changed 17 months ago by robe

yes and it is available in final distribution. We have this silly discussion every release.

Changed 17 months ago by robe

Changed 17 months ago by strk

I probably wasn't there during discussion days. Do you have a pointer to mailing list ? What else do you want in tarball ? .pdf manual ? html manual ?

Changed 17 months ago by robe

The pdf is 5 MB and its already linked on the documentation page  http://www.postgis.org/documentation/

so see no point in including it. It seems to be included in our incremental builds but not in final which is even more illogical.

html don't really care either way because I never bother including it in windows distribution because there are too many moving pieces. So just have a local link to the PDF and the web site html if people really want an html version.

Changed 17 months ago by strk

  • status changed from assigned to closed
  • resolution set to fixed

Let's see if you like r8716. Should ship comments and no manual (neither PDF nor HTML).

Note: See TracTickets for help on using tickets.