Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix compilation with freepascal #17

Open
wants to merge 12 commits into
base: master
Choose a base branch
from

Conversation

blikblum
Copy link

This pull request fixes compilation with Lazarus and fpc 2.6.4. Currently it compiles and works only in Windows since some units depend heavily on win api. The sources changes are small and i took care to avoid breaking Delphi compilation, but since i don't have access to Delphi i cannot test.

Changes:

  • Remove buggy and unnecessary GetMem custom implementation
  • Wrap Delphi specific code in defines
  • Adapt packages to avoid path conflicts and to properly compile as needed
  • Merge JclContainers package into Fcl since there's a circular dependency between both
  • Update gitignore to match fpc / Lazarus created files
  • Include ini file by default. The jcl install script is not necessary at all for Lazarus / fpc
  • Adapt ExprEval example to Lazarus

blikblum and others added 12 commits November 15, 2014 15:09
* Add TStringHash class
* Add missing overloads
* Enclose unused function of zlibh in a define
* Remove JclBase.GetMem: gives an infinite recursion error. Also is
unnecessary since fpc is using out now and
* The package compiles now as needed
* Added missing files
* Removed unnecessary options
* Add more fpc/Lazarus files to gitignore
* Add more specific rules in gitignore so all lib files created by
Lazarus are ignored
* Added jclfpc.inc. The installation process of a Lazarus package does
not require a install script
Make ExprEval example compatible with Lazarus
* Merge JclContainers and Jcl packages
* Fix container compilation (workaround to fpc compiler bug)
* Fix compilation of JclVcl and and DeveloperTools (only for windows)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant