Sign in Join

Live chat for Free Pascal, Lazarus, Delphi and every Pascal programmer.

Mailing lists / lazarus / 2012-07

Lazarus debugger location on Win x86/x64

23 messages · 4 people · –

Reply by email Subscribe to this list Your subscription

Read-only archive. Use j and k to move between messages.

Reinier Olislagers

Hi list,

http://bugs.freepascal.org/view.php?id=22086
indicates macros are supported in the debugger path.

The location for gdb.exe in EnvironmentOptions.xml is set by default to <DebuggerFilename Value="%LazDir%\mingw\bin\gdb.exe">

Would it make sense to create this directory structure %LazDir%\mingw\bin\$(TargetCPU)\
or even
%LazDir%\mingw\bin\$(TargetCPU)-$(TargetOS)

This way, we could change the debugger depending on project target, which helps if you have e.g. both x86 and x64 LCL installed: <DebuggerFilename Value="%LazDir%\mingw\bin\%TargetCPU%-%TargetOS%\gdb.exe">

Only thing is: I don't know if TargetCPU and TargetOS are actually supported: I tried looking for LazDir to see if it is an IDE macro (which it presumably is??) in
http://wiki.lazarus.freepascal.org/IDE_Macros_in_paths_and_filenames and didn't find anything.

Regards,
Reinier

Reply by email

Mattias Gaertner
to Reinier Olislagers
Hi list, http://bugs.freepascal.org/view.php?id=22086 indicates macros are supported in the debugger path.

yes.

The location for gdb.exe in EnvironmentOptions.xml is set by default to <DebuggerFilename Value="%LazDir%\mingw\bin\gdb.exe">

The %LazDir% is replaced by the installer. It should not be in the installed file.

Would it make sense to create this directory structure %LazDir%\mingw\bin\$(TargetCPU)\ or even %LazDir%\mingw\bin\$(TargetCPU)-$(TargetOS)

I don't know mingw, so I can't help there.

Show quoted text 10 lines
This way, we could change the debugger depending on project target, which helps if you have e.g. both x86 and x64 LCL installed: <DebuggerFilename Value="%LazDir%\mingw\bin\%TargetCPU%-%TargetOS%\gdb.exe"> Only thing is: I don't know if TargetCPU and TargetOS are actually supported: I tried looking for LazDir to see if it is an IDE macro (which it presumably is??) in http://wiki.lazarus.freepascal.org/IDE_Macros_in_paths_and_filenames and didn't find anything.

The macro is $(LazarusDir).

Mattias

Reply by email

Reinier Olislagers
to Mattias Gaertner
Mattias Gaertner wrote 7 lines
On Mon, 30 Jul 2012 10:38:57 +0200 Reinier Olislagers wrote: > The location for gdb.exe in EnvironmentOptions.xml is set by default to > <DebuggerFilename Value="%LazDir%\mingw\bin\gdb.exe"> The %LazDir% is replaced by the installer. It should not be in the installed file.

Ok, got that.

Show quoted text 6 lines
> Would it make sense to create this directory structure > %LazDir%\mingw\bin\$(TargetCPU)\ > or even > %LazDir%\mingw\bin\$(TargetCPU)-$(TargetOS) I don't know mingw, so I can't help there.

Neither do I... but there is a 64 bit Windows gdb.exe that is different from the 32 bit Windows gdb.exe.
On the off chance that debuggers might become (or are) available for other platforms (e.g. wince-arm) perhaps the second notation would be the best.

Regarding the naming: I obviously chose those to correspond to the macros in the IDE. If debugger executable names for different platforms would not be called gdb.exe, further adjustments would need to be made... but AFAIU they couldn't be covered by a simple macro in the debugger path.
(I also vaguely recall that mingw or at least gcc has a different naming convention for platforms/CPUs... which will not influence the issue above but could impact other systems)

For clarity: I'm referring to the Windows installer tree here, so no effect for other platforms.

Alternatives could be:
1. leave the situation as is
2. create a debugger path setting in the GUI per targetos/cpu pair. Seems complicated given the Win32/Win64 goal described above but might perhaps be useful when setting up cross compiling environments on Linux... no experience there though.

Show quoted text 11 lines
> This way, we could change the debugger depending on project target, > which helps if you have e.g. both x86 and x64 LCL installed: > <DebuggerFilename Value="%LazDir%\mingw\bin\%TargetCPU%-%TargetOS%\gdb.exe"> > > Only thing is: I don't know if TargetCPU and TargetOS are actually > supported: I tried looking for LazDir to see if it is an IDE macro > (which it presumably is??) in > http://wiki.lazarus.freepascal.org/IDE_Macros_in_paths_and_filenames > and didn't find anything. The macro is $(LazarusDir).

Yep, thanks, that makes sense - I was thrown by the LazDir in the installer's version of environmentoptions.xml :)

Reply by email

Ludo Brands
to Reinier Olislagers
Show quoted text 11 lines
>> Would it make sense to create this directory structure >> %LazDir%\mingw\bin\$(TargetCPU)\ or even >> %LazDir%\mingw\bin\$(TargetCPU)-$(TargetOS) > > I don't know mingw, so I can't help there. Neither do I... but there is a 64 bit Windows gdb.exe that is different from the 32 bit Windows gdb.exe. On the off chance that debuggers might become (or are) available for other platforms (e.g. wince-arm) perhaps the second notation would be the best.

The wince debugger is also called gdb.exe and having a %LazDir%\mingw\bin\$(TargetCPU)-$(TargetOS) default debug path would indeed be useful.

Ludo

Reply by email

Martin
to Ludo Brands
Ludo Brands wrote 14 lines
>>> Would it make sense to create this directory structure >>> %LazDir%\mingw\bin\$(TargetCPU)\ or even >>> %LazDir%\mingw\bin\$(TargetCPU)-$(TargetOS) >> I don't know mingw, so I can't help there. > Neither do I... but there is a 64 bit Windows gdb.exe that is > different from the 32 bit Windows gdb.exe. On the off chance > that debuggers might become (or are) available for other > platforms (e.g. wince-arm) perhaps the second notation would > be the best. > The wince debugger is also called gdb.exe and having a %LazDir%\mingw\bin\$(TargetCPU)-$(TargetOS) default debug path would indeed be useful.

There also is the idea, of having a per-project debugger config, that can override any of that.

But if ide-macros are already supported, any one who needs, can test change it

Reply by email

Reinier Olislagers
to Martin
Martin wrote 21 lines
On 30/07/2012 10:57, Ludo Brands wrote: >>>> Would it make sense to create this directory structure >>>> %LazDir%\mingw\bin\$(TargetCPU)\ or even >>>> %LazDir%\mingw\bin\$(TargetCPU)-$(TargetOS) >>> I don't know mingw, so I can't help there. >> Neither do I... but there is a 64 bit Windows gdb.exe that is >> different from the 32 bit Windows gdb.exe. On the off chance >> that debuggers might become (or are) available for other >> platforms (e.g. wince-arm) perhaps the second notation would >> be the best. >> > The wince debugger is also called gdb.exe and having a > %LazDir%\mingw\bin\$(TargetCPU)-$(TargetOS) default debug path would > indeed > be useful. > There also is the idea, of having a per-project debugger config, that can override any of that. But if ide-macros are already supported, any one who needs, can test change it

Ok - patch submitted for the environment setting:

http://bugs.freepascal.org/view.php?id=22533

This also needs changes for the Windows source directory that is used to build the installer...

Thanks,
Reinier

Reply by email

Ludo Brands
to Martin: There also is the idea, of having a per-project debugger config, that can override any of…
But if ide-macros are already supported, any one who needs, can test change it

Sure, but the request is to have this as default in Lazarus which makes supporting multiple targets a little bit easier. It obviously requires changing the directory structure in the lazarus distributions.

CodeTyphon is doing something similar since they want to be multi target out of the box.

Ludo

Reply by email

Martin
to Reinier Olislagers: Ok - patch submitted for the environment setting:…
Reinier Olislagers wrote 7 lines
Ok - patch submitted for the environment setting: http://bugs.freepascal.org/view.php?id=22533 This also needs changes for the Windows source directory that is used to build the installer...

Not sure what you mean by this.

Also not sure why you patch the RemoveFiles, if you dont batch the rest of the installer files?
Changes probably have to go into the lazarus.iss file (maybe, but probably not, the batch)

-------
In any case, the RC (coming soon) have been build with the current structure. So the current structure will be presented for testing. That means it is not wise to change the structure before 1.0, because then the new structure can not be tested any more

Reply by email

Martin
to Reinier Olislagers: Ok - patch submitted for the environment setting:…
Reinier Olislagers wrote 6 lines
http://bugs.freepascal.org/view.php?id=22533 This also needs changes for the Windows source directory that is used to build the installer...

There also will be a need for testing upgrades. If the current setting points to the none dependent location.

Then what happens if you upgrade? gdb is removed, and moved else where. But the old config still exists. The user will be presented with an invalid config.

MAybe the setup dialog can be extended.... I don't know.

Reply by email

Martin
to Martin: Not sure what you mean by this. Also not sure why you patch the RemoveFiles, if you dont…
Martin wrote 15 lines
On 30/07/2012 12:28, Reinier Olislagers wrote: > Ok - patch submitted for the environment setting: > > http://bugs.freepascal.org/view.php?id=22533 > > This also needs changes for the Windows source directory that is used to > build the installer... > Not sure what you mean by this. Also not sure why you patch the RemoveFiles, if you dont batch the rest of the installer files? Changes probably have to go into the lazarus.iss file (maybe, but probably not, the batch)

The new location can be set in the create_installer.bat The line below currently does an svn export (from the laz binaries svn). It can be changed. The bin directory is part of the svn, so the platform would go mingw/winxxx-xxx/ (bin/gdb.exe)

:: copy gdb into build dir
if exist %GDBDIR% %SVN% export %GDBDIR% %BUILDDIR%\mingw

I can deal with the installer. but there first must be a solution for upgrading

Reply by email

Reinier Olislagers
to Martin: Not sure what you mean by this. Also not sure why you patch the RemoveFiles, if you dont…
Martin wrote 10 lines
On 30/07/2012 12:28, Reinier Olislagers wrote: > Ok - patch submitted for the environment setting: > > http://bugs.freepascal.org/view.php?id=22533 > > This also needs changes for the Windows source directory that is used to > build the installer... > Not sure what you mean by this.

The Lazarus\tools\install\win\lazarus.iss Inno Setup file refers to Source: {#BuildDir}\*.*; DestDir: {app}; Flags: recursesubdirs to presumably get its source files.
I have no gdb.exe in my svn checkout and therefore suppose the build directory in question is generated in another way... from some sort of source directory.
... see below though - I think changing create_installer.bat should be sufficient

Also not sure why you patch the RemoveFiles, if you dont batch the rest of the installer files?

You mean why do I remove gdb.exe from the old position while not adding it to the new position?
See above: I didn't understand how to do that.

Changes probably have to go into the lazarus.iss file (maybe, but probably not, the batch)

Ah got you, create_installer.bat should indeed be changed. (Just read your other mail about that)

So changing lines 110-111:
:: copy gdb into build dir
if exist %GDBDIR% %SVN% export %GDBDIR% %BUILDDIR%\mingw into
:: copy gdb below the build dir for the proper os/cpu combination if exist %GDBDIR% %SVN% export %GDBDIR% %BUILDDIR%\mingw\%FPCFULLTARGET%\

would mean we'd get e.g.
<lazarus>\mingw\i386-win32\bin\gdb.exe
or
<lazarus>\mingw\x86_64-win64\bin\gdb.exe
Right?

In any case, the RC (coming soon) have been build with the current structure. So the current structure will be presented for testing. That means it is not wise to change the structure before 1.0, because then the new structure can not be tested any more

I don't mind when it changes because I'm on SVN anyway.

I would welcome a change in the post 1.0 branch though, because:
1. this seems to be useful functionality for not much added effort
2. I'm adjusting fpcup already and would like to stay as close as the official installers as possible
-------------- next part --------------
Index: tools/install/win/create_installer.bat
=================================================================== --- tools/install/win/create_installer.bat (revision 38091) +++ tools/install/win/create_installer.bat (working copy) @@ -107,8 +107,8 @@
if not exist %BUILDDIR%\lazarus.exe goto END
if not exist %BUILDDIR%\startlazarus.exe goto END

-:: copy gdb into build dir
-if exist %GDBDIR% %SVN% export %GDBDIR% %BUILDDIR%\mingw +:: copy gdb below the build dir for the proper os/cpu combination +if exist %GDBDIR% %SVN% export %GDBDIR% %BUILDDIR%\mingw\bin\%FPCFULLTARGET%\

:: create the installer
IF [%BUILDLAZRELEASE%]==[] GOTO SNAPSHOT

Reply by email

Reinier Olislagers
to Martin: The new location can be set in the create_installer.bat The line below currently does an…
Martin wrote 25 lines
On 30/07/2012 13:31, Martin wrote: > On 30/07/2012 12:28, Reinier Olislagers wrote: >> Ok - patch submitted for the environment setting: >> >> http://bugs.freepascal.org/view.php?id=22533 >> >> This also needs changes for the Windows source directory that is used to >> build the installer... >> > > Not sure what you mean by this. > > Also not sure why you patch the RemoveFiles, if you dont batch the > rest of the installer files? > Changes probably have to go into the lazarus.iss file (maybe, but > probably not, the batch) The new location can be set in the create_installer.bat The line below currently does an svn export (from the laz binaries svn). It can be changed. The bin directory is part of the svn, so the platform would go mingw/winxxx-xxx/ (bin/gdb.exe) :: copy gdb into build dir if exist %GDBDIR% %SVN% export %GDBDIR% %BUILDDIR%\mingw

Thanks, see my other mail.

I can deal with the installer. but there first must be a solution for upgrading

I'd suggest we
1. remove existing gdb.exe from the old location
2. check if the setting in (the default?) primary config path points to the removed gdb.exe
3. change it to the new location

We could probably reuse some code that was used during the last big settings upgrade (0.9.31? can't remember)

Disadvantage: people with custom gdb.exe will not have it changed

Alternatively, we can just overwrite the user's debugger setting, but that seems a bit unfriendly as they already changed the default and presumably know what they're doing.

In both cases, non-default configs won't be touched, but I don't see any realistic way to address that.

Regards,
Reinier

Reply by email

Martin
to Reinier Olislagers: The Lazarus\tools\install\win\lazarus.iss Inno Setup file refers to Source:…
Reinier Olislagers wrote
I have no gdb.exe in my svn checkout and therefore suppose the build directory in question is generated in another way... from some sort of source directory.

http://svn.freepascal.org/svn/lazarus/binaries

you can use the tortoise svn repro browser, and start at http://svn.freepascal.org/svn/lazarus

Show quoted text 6 lines
So changing lines 110-111: :: copy gdb into build dir if exist %GDBDIR% %SVN% export %GDBDIR% %BUILDDIR%\mingw into :: copy gdb below the build dir for the proper os/cpu combination if exist %GDBDIR% %SVN% export %GDBDIR% %BUILDDIR%\mingw\%FPCFULLTARGET%\

yes

Reply by email

Martin
to Reinier Olislagers: Thanks, see my other mail. I'd suggest we 1. remove existing gdb.exe from the old location

Personally I believe extending the setup (as described below) is a must, in order to change the installer.

Reinier Olislagers wrote 10 lines
> I can deal with the installer. but there first must be a solution for > upgrading I'd suggest we 1. remove existing gdb.exe from the old location 2. check if the setting in (the default?) primary config path points to the removed gdb.exe 3. change it to the new location We could probably reuse some code that was used during the last big settings upgrade (0.9.31? can't remember)

That it was the setup dialog does (for fpc, fpc sources, and lazarus dir) Run
lazarus.exe --setup

Issue is, I don't have the time to should:
- Check the current config, use it, if it appears ok (potentially run it, to get the version)
- If there is no environmentoptions.xml in the primary conf, then it uses secondary conf (the environmentoptions.xml in the exe dir)
- Look in a list of default locations

If it changes existing primary conf, then it should always inform the user

Disadvantage: people with custom gdb.exe will not have it changed

The must have it in a none default location (or it would be overwritten). So that is simply kept. I see no disadvantage. But the setup can give them a change to notice the new structure.

Maybe the new structure can also be added to the list of know gdb in the drop down.

However, the setup dialog is the same for all platforms. So the list of path to look must be included for the various platforms.

Alternatively, we can just overwrite the user's debugger setting, but that seems a bit unfriendly as they already changed the default and presumably know what they're doing.

No way

Reply by email

Reinier Olislagers
to Martin
Martin wrote 16 lines
Personally I believe extending the setup (as described below) is a must, in order to change the installer. On 30/07/2012 14:25, Reinier Olislagers wrote: >> I can deal with the installer. but there first must be a solution >> for upgrading > I'd suggest we 1. remove existing gdb.exe from the old location 2. > check if the setting in (the default?) primary config path points > to the removed gdb.exe 3. change it to the new location > > We could probably reuse some code that was used during the last > big settings upgrade (0.9.31? can't remember) That it was the setup dialog does (for fpc, fpc sources, and lazarus dir) Run lazarus.exe --setup

Are you sure? I seem to remember it was a different dialog that ran once after the Inno setup finished: it mentioned old version detected, and would I like to migrate to the new version... but who knows it could have been kicked off by the Lazarus setup dialog checks.

Issue is, I don't have the time to problem. It should: - Check the current config, use it, if it appears ok (potentially run it, to get the version)

Yes, it should check if the gdb.exe in the debugger file exists if the setting points to the old location (<lazdir>\mingw\bin\gdb.exe). If it does, it doesn't need to do anything.
If it doesn't exist, there's been an upgrade (or the user removed gdb.exe himself)

- If there is no environmentoptions.xml in the primary conf, then it uses secondary conf (the environmentoptions.xml in the exe dir)

This check is already in place, isn't it?

- Look in a list of default locations

If we're on Windows, the config pointed to the old location and gdb.exe does not exist there anymore, then we can just change it to the new location if gdb.exe exists there.
If it isn't in the new location, the user can figure it out, the same as now, because he obviously changed something himself.

Of course, if this default locations list already exists, as well as check to assign the debugger, adapting that is probably easier.

If it changes existing primary conf, then it should always inform the user

Yes.

> Disadvantage: people with custom gdb.exe will not have it changed The must have it in a none default location (or it would be overwritten). So that is simply kept. I see no disadvantage.

If you're happy, I'm happy.

But the setup can give them a change to notice the new structure. Maybe the new structure can also be added to the list of know gdb in the drop down.

You mean the history list? I don't really understand what you mean here.

However, the setup dialog is the same for all platforms. So the list of path to look must be included for the various platforms.

See above for my suggestions on testing for Windows only. On Linux/Unix/OSX, running which gdb would probably be the easiest way to get a suggestion for a location.

> Alternatively, we can just overwrite the user's debugger setting, > but that seems a bit unfriendly as they already changed the default > and presumably know what they're doing. No way

Glad you agree ;)

Reply by email

Martin
to Reinier Olislagers
Reinier Olislagers wrote 9 lines
On 30-7-2012 17:11, Martin wrote: >>> We could probably reuse some code that was used during the last >>> big settings upgrade (0.9.31? can't remember) > That it was the setup dialog does (for fpc, fpc sources, and lazarus > dir) Run lazarus.exe --setup Are you sure? I seem to remember it was a different dialog that ran once after the Inno setup finished: it mentioned old version detected, and would I like to migrate to the new version... but who knows it could have been kicked off by the Lazarus setup dialog checks.

If the configuration has no conflicting/missing values, it will simply show the upgrade info.
But if some values are missing then the setup dialog comes...

Afaik it is all connected. But then, I am not sure.

Show quoted text 11 lines
> It should: - Check the current config, use it, if it appears ok > (potentially run it, to get the version) Yes, it should check if the gdb.exe in the debugger file exists if the setting points to the old location (<lazdir>\mingw\bin\gdb.exe). If it does, it doesn't need to do anything. If it doesn't exist, there's been an upgrade (or the user removed gdb.exe himself) > - If there is no environmentoptions.xml in the primary conf, then it > uses secondary conf (the environmentoptions.xml in the exe dir) This check is already in place, isn't it?

Afaik yes. I only looked briefly at it. (some while ago)

> - Look in a list of default locations If we're on Windows, the config pointed to the old location and gdb.exe does not exist there anymore, then we can just change it to the new location if gdb.exe exists there.

Yes, if that is easy to support...

I do not maintain the auto setup code. Hard to say what can or should or must be there. (except for the obvious: a copy of what is currently there for ather parts of the setup. eg. fpc dir)

Show quoted text 5 lines
> But the setup can give them a change to notice the new structure. > > Maybe the new structure can also be added to the list of know gdb in > the drop down. You mean the history list? I don't really understand what you mean here.

The gdb location in the opt dialog is a drop down with previous values in it.

Btw, if you install from svn, it currently sets something there too. And that is taken from the debugger units.... But that may need to be adapted too
-----
Sorry
I cant be to much help on that setup issue at the moment.

If I could, I could probably fix it very quickly myself...

Reply by email

Reinier Olislagers
to Martin
Martin wrote 55 lines
On 30/07/2012 16:30, Reinier Olislagers wrote: > On 30-7-2012 17:11, Martin wrote: >>>> We could probably reuse some code that was used during the last >>>> big settings upgrade (0.9.31? can't remember) >> That it was the setup dialog does (for fpc, fpc sources, and lazarus >> dir) Run lazarus.exe --setup > Are you sure? I seem to remember it was a different dialog that ran once > after the Inno setup finished: it mentioned old version detected, and > would I like to migrate to the new version... but who knows it could > have been kicked off by the Lazarus setup dialog checks. If the configuration has no conflicting/missing values, it will simply show the upgrade info. But if some values are missing then the setup dialog comes... Afaik it is all connected. But then, I am not sure. >> It should: - Check the current config, use it, if it appears ok >> (potentially run it, to get the version) > Yes, it should check if the gdb.exe in the debugger file exists if the > setting points to the old location (<lazdir>\mingw\bin\gdb.exe). > If it does, it doesn't need to do anything. > If it doesn't exist, there's been an upgrade (or the user removed > gdb.exe himself) > >> - If there is no environmentoptions.xml in the primary conf, then it >> uses secondary conf (the environmentoptions.xml in the exe dir) > This check is already in place, isn't it? Afaik yes. I only looked briefly at it. (some while ago) >> - Look in a list of default locations > If we're on Windows, the config pointed to the old location and gdb.exe > does not exist there anymore, then we can just change it to the new > location if gdb.exe exists there. Yes, if that is easy to support... I do not maintain the auto setup code. Hard to say what can or should or must be there. (except for the obvious: a copy of what is currently there for ather parts of the setup. eg. fpc dir) >> But the setup can give them a change to notice the new structure. >> >> Maybe the new structure can also be added to the list of know gdb in >> the drop down. > You mean the history list? I don't really understand what you mean here. The gdb location in the opt dialog is a drop down with previous values in it. Btw, if you install from svn, it currently sets something there too. And that is taken from the debugger units.... But that may need to be adapted too ----- Sorry I cant be to much help on that setup issue at the moment. If I could, I could probably fix it very quickly myself...

No problem, Martin, as long as what I deliver is more or less what you expect ;)
Thanks for the help!

Reply by email

Martin
to Reinier Olislagers
Reinier Olislagers wrote 13 lines
Btw, if you install from svn, it currently sets something there too. And that is taken from the debugger units.... But that may need to be adapted too ----- Sorry I cant be to much help on that setup issue at the moment. If I could, I could probably fix it very quickly myself... No problem, Martin, as long as what I deliver is more or less what you expect ;) Thanks for the help!

The think is, I had some info on the whole setup dlg (thanks to Mattias), because I did once had plans to add gdb there anyway. But I cant find that mail anymore....

Reply by email

Reinier Olislagers
to Martin
Martin wrote 18 lines
On 30/07/2012 17:16, Reinier Olislagers wrote: > Btw, if you install from svn, it currently sets something there too. And > that is taken from the debugger units.... But that may need to be > adapted too > ----- > Sorry > I cant be to much help on that setup issue at the moment. > > If I could, I could probably fix it very quickly myself... > No problem, Martin, as long as what I deliver is more or less what you > expect ;) > Thanks for the help! > > The think is, I had some info on the whole setup dlg (thanks to Mattias), because I did once had plans to add gdb there anyway. But I cant find that mail anymore....

Ok, finished.

That --setup dialog code is nice - even I could understand it ;)

I extended the setup checks:
- make.exe check was not visible and changed config without asking. Created additional tab and made it visible.
- added tab and checks for debugger.
Limited testing (didn't test all possible file locations) on Win32: works fine.
Patch in issue 22539 (perhaps Mattias would like to have a look at that one?)

After that is implemented, I think we're good to go for the patches in 22533

Thanks,
Reinier

Reply by email

Reinier Olislagers
to Martin: The new location can be set in the create_installer.bat The line below currently does an…
Martin wrote 15 lines
On 30/07/2012 13:31, Martin wrote: > On 30/07/2012 12:28, Reinier Olislagers wrote: >> Ok - patch submitted for the environment setting: The new location can be set in the create_installer.bat The line below currently does an svn export (from the laz binaries svn). It can be changed. The bin directory is part of the svn, so the platform would go mingw/winxxx-xxx/ (bin/gdb.exe) :: copy gdb into build dir if exist %GDBDIR% %SVN% export %GDBDIR% %BUILDDIR%\mingw I can deal with the installer. but there first must be a solution for upgrading

Martin, (and anybody else interested)

As you may know, the upgrade --setup dialog patch has been committed (and improved).

I've finally patched the installer creator script correctly to match the current --setup dialog default locations... I think & you can check ;)

Tested it with a Win x86, Laz SVN today and FPC 2.6.1 installer.

Also added some more details to the readme (e.g. CHM help file parametes), some typos in create_setup if hitting empty parameters etc.

Single patch available at:
http://bugs.freepascal.org/view.php?id=22533

Could somebody have a look?

Thanks,
Reinier

Reply by email

Martin
to Reinier Olislagers
Reinier Olislagers wrote 18 lines
Martin, (and anybody else interested) As you may know, the upgrade --setup dialog patch has been committed (and improved). I've finally patched the installer creator script correctly to match the current --setup dialog default locations... I think & you can check ;) Tested it with a Win x86, Laz SVN today and FPC 2.6.1 installer. Also added some more details to the readme (e.g. CHM help file parametes), some typos in create_setup if hitting empty parameters etc. Single patch available at: http://bugs.freepascal.org/view.php?id=22533 Could somebody have a look?

I already assigned it to me.

As far as the gdb location goes, I will look at it, but it may have to wait until after 1.0. It is easier for me at the moment to keep the installer in trunk and fixes the same, that way I can test fixes build using my trunk checkout, and then later merge.....

If there are other changes, I will have a look, but individual patches will usually make it easier.

On top of that, I have one or two actual bugs, that I want to solve first (because the may be candidates for being merged)

Reply by email

Reinier Olislagers
to Martin
Martin wrote 26 lines
On 02/08/2012 22:05, Reinier Olislagers wrote: > Martin, (and anybody else interested) > > As you may know, the upgrade --setup dialog patch has been committed > (and improved). > > I've finally patched the installer creator script correctly to match the > current --setup dialog default locations... I think & you can check ;) > > Tested it with a Win x86, Laz SVN today and FPC 2.6.1 installer. > > Also added some more details to the readme (e.g. CHM help file > parametes), some typos in create_setup if hitting empty parameters etc. > > Single patch available at: > http://bugs.freepascal.org/view.php?id=22533 > > Could somebody have a look? > I already assigned it to me. As far as the gdb location goes, I will look at it, but it may have to wait until after 1.0. It is easier for me at the moment to keep the installer in trunk and fixes the same, that way I can test fixes build using my trunk checkout, and then later merge.....

Fine by me.

If there are other changes, I will have a look, but individual patches will usually make it easier.

It's not much (mostly cosmetic and typos), but I'd be happy to split it up.

On top of that, I have one or two actual bugs, that I want to solve first (because the may be candidates for being merged)

Ok.

Just let me know if you want me to redo the patch after a certain revision (or split it up).

Thanks,
Reinier

Reply by email

Martin
to Reinier Olislagers
Reinier Olislagers wrote 8 lines
On 2-8-2012 23:25, Martin wrote: > On top of that, I have one or two actual bugs, that I want to solve > first (because the may be candidates for being merged) Ok. Just let me know if you want me to redo the patch after a certain revision (or split it up).

As soon as I had the time to look at it.

Reply by email