View Issue Details

IDProjectCategoryView StatusLast Update
0005598JEDI VCL02 Installationpublic2012-09-10 14:15
ReporterGeroldAssigned ToAHUser 
PrioritynormalSeveritymajorReproducibilityalways
Status resolvedResolutionfixed 
Product VersionDaily / GIT 
Target VersionFixed in Version3.46 
Summary0005598: D6 Installation fails with JVCLThemesEnabled in JvExplorerBarItems
DescriptionCannot install latest JVCL under Delphi 6 with "XP Theming" enabled in the JVCL 3.41 installer.

In JvExplorerBarItems.pas I found:

...
implementation

uses
  {$IFDEF JVCLThemesEnabled}
  Themes,
  {$ENDIF JVCLThemesEnabled}
  Math, DateUtils;
...

There exists no "Themes.pas" in Delphi 6.

Guess it should be:

  {$IFDEF JVCLThemesEnabled}
  {$IFDEF COMPILER7_UP}
  Themes,
  {$ELSE}
  ThemeSrv,
  {$ENDIF COMPILER7_UP}
  {$ENDIF JVCLThemesEnabled}
  Math, DateUtils;

or

  {$IFDEF JVCLThemesEnabled}
  {$IFDEF COMPILER7_UP}
  Themes,
  {$ENDIF COMPILER7_UP}
  {$ENDIF JVCLThemesEnabled}
  Math, DateUtils;

TagsNo tags attached.

Activities

obones

2011-06-20 11:47

administrator   ~0018787

The readme.htm file says that if you want theming for D6, you MUST install Mike Lischke's ThemeManager package.
Have you done so?

Gerold

2011-06-20 12:09

reporter   ~0018788

Yes of course. As I said before, there exists no "Themes.pas" in my D6 ENT or in Mike Lischke's Thememanger.

You can find the objects etc. defined in "Themes.pas" from D7 (or higher???) in "ThemeSrv.pas" from ML's ThemeManager. So I think the units definied between
{$IFDEF JVCLThemesEnabled} and {$ENDIF JVCLThemesEnabled} must be changed for D6.

BTW:
I dont use "XP Theming" in D6 anymore. It was just a test, but maybe other users will have the same problem.

obones

2011-09-21 11:47

administrator   ~0018922

Need to test with D6 + Theming

AMarchant

2011-10-28 03:18

reporter   ~0019058

Hi: just to confirm that this issue still exists as of revision 13104 (Delphi 6) - but I think the source has shifted - although the error I'm getting:

"C:\Program Files\Borland\Delphi6\Source\3rdParty\JEDI\JVCL\run\JvExplorerBarItems.pas(398) Fatal: File not found: 'Themes.dcu'"

...isn't entirely consistent with that I think is the cause. JvExplorerBarItems.pas "uses" JvExplorerBar.pas, and like JvExplorerBar of old, it contains:

{$IFDEF JVCLThemesEnabled}
Themes,
{$ENDIF JVCLThemesEnabled}

...in its uses clause. Perhaps, as the OP suggested before, this should conditionally be ThemeSrv for <= Delphi6 with ThemeManager installed?

AMarchant

2011-10-28 03:27

reporter   ~0019059

Update: sorry, my mistake, I mis-read the first message - it's very late and I've been trying to get around these little niggly issues for hours.

The installer reports the error quoted in my last message, it doesn't involve JvExplorerBar.pas, I'm not sure how I managed to muddle that up - the problem is with JvExplorerBarItems.pas, and its "uses" of "Themes", which matches no named unit in the standalone version of ThemeManager for Delphi 4-6.

AMarchant

2011-10-28 16:52

reporter   ~0019060

With the benefit of a less sleep addled-mind, I modified JvExplorerBarItems.pas, switching:

397 {$IFDEF JVCLThemesEnabled}
398 Themes,
399 {$ENDIF JVCLThemesEnabled}

to

397 {$IFDEF JVCLThemesEnabled}
398 JvThemes,
399 {$ENDIF JVCLThemesEnabled}

...to match all other controls, and it then compiled without error.

jfudickar

2011-10-28 22:03

developer   ~0019065

Changed in SVN.

Please retest

AMarchant

2011-10-29 15:05

reporter   ~0019067

Tested - that compiles without error now, thank you.

grubste

2011-10-29 20:06

reporter   ~0019068

That change does not compile with Delphi XE2 now.
I had to change JvThemes to Themes again.

AHUser

2011-11-02 21:10

developer   ~0019079

XE2 compilation is restored.

Issue History

Date Modified Username Field Change
2011-06-16 10:23 Gerold New Issue
2011-06-20 11:47 obones Note Added: 0018787
2011-06-20 11:47 obones Status new => feedback
2011-06-20 12:09 Gerold Note Added: 0018788
2011-09-21 11:47 obones Note Added: 0018922
2011-09-21 11:47 obones Status feedback => new
2011-09-21 14:15 obones Status new => acknowledged
2011-10-28 03:18 AMarchant Note Added: 0019058
2011-10-28 03:27 AMarchant Note Added: 0019059
2011-10-28 16:52 AMarchant Note Added: 0019060
2011-10-28 22:03 jfudickar Note Added: 0019065
2011-10-28 22:03 jfudickar Status acknowledged => feedback
2011-10-29 15:05 AMarchant Note Added: 0019067
2011-10-29 20:06 grubste Note Added: 0019068
2011-11-02 21:10 AHUser Note Added: 0019079
2011-11-02 21:10 AHUser Status feedback => resolved
2011-11-02 21:10 AHUser Fixed in Version => Daily / SVN
2011-11-02 21:10 AHUser Resolution open => fixed
2011-11-02 21:10 AHUser Assigned To => AHUser
2012-09-10 14:15 obones Fixed in Version Daily / SVN => 3.46