Item8769: pseudo-install needs to create/linked derived objects in the core

pencil
Priority: Normal
Current State: Closed
Released In: 1.1.0
Target Release: minor
Applies To: Extension
Component: BuildContrib
Branches:
Reported By: CrawfordCurrie
Waiting For:
Last Change By: FlorianSchlichting
There's currently no way for pseuod-install to handle derived objects in the core. For example, if I have a .js file in the core there's no way for that to be linked to the corresponding _src.js file.

The simplest way to deal with this is for pseudo-install to process the Contrib/core/MANIFEST, by adding core as a module that can be installed. This module should be auto-installed by default, developer and all targets as well.

Marked as urgent because as long as we don't have this, we have to have these generated .js files checked in, which can cause major confusion when you are developing.

-- CrawfordCurrie - 25 Mar 2010

Done

-- CrawfordCurrie - 25 Mar 2010

I'm seeing errors building, while running pseudo-install.pl -clean -A developer as part of the build, and guessing this change is where they started.

mkdir AUTHORS: File exists at pseudo-install.pl line 250

Adding a print statement to copy_in, the arguments passed to it were:
copy_in(., AUTHORS, AUTHORS, 1)

AUTHORS is the first entry in core/lib/Foswiki/Contrib/core/MANIFEST

-- DrakeDiedrich - 26 Mar 2010

  1. pseudo-install.pl has no -clean parameter
  2. what platform were you building on? copy_in will only be called on platforms that don't support soft-links

AUTHORS is not a derived object, and it's unlikely to be related to this change, but I can't rule it out.

-- CrawfordCurrie - 30 Mar 2010

I'm running into the same problem. It appears that pseudo-install -copy default is trying to copy core on top of itself. If there is no directory for the file, the filename is also passed to copy_in as the directory name, so for Authors, $dir = AUTHORS, $file = AUTHORS, and it calls mkpath for AUTHORS.

I'm trying to build a local release, so -copy is used to force copying of the files. Linking is not done according to BuildingARelease

-- GeorgeClark - 10 Apr 2010

I think I've found a fix for the -copy issue. It works here but needs validation from someone more familiar with pseudo-install.pl

-- GeorgeClark - 10 Apr 2010

Seems to work; I didn't inspect the code, though.

-- CrawfordCurrie - 05 May 2010

http://debmarshal.debian.net/foswiki/logs/

Autobuilds started succeeding again shortly after this patch in r7186, so it looks like everything needed to build .debs again is in trunk.

-Drake

-- DrakeDiedrich - 05 May 2010
 
Topic revision: r16 - 29 Nov 2013, FlorianSchlichting
The copyright of the content on this website is held by the contributing authors, except where stated elsewhere. See Copyright Statement. Creative Commons License    Legal Imprint    Privacy Policy