[Pkg-pascal-devel] are we waiting on each other?

[Abou Al Montacir]

Hi Paul,

Sorry for my late answer, but generally I'm short in time the WE. So do
not hesitate to take decisions. Anyway any commit can always be reverted
if needed ;).

On Fri, 2014-06-06 at 21:57 +0200, Paul Gevers wrote:
> On 26-05-14 23:01, Abou Al Montacir wrote:
> > I'll contact upstream to request fixing this issue.
> 
> Did you contact Michalis?
No, but I should indeed.

> >                                                     I don't like the
> > dependency on sources. If upstream consider this include file to be
> > included bu third party code then we shall move it to fp-units-*
> 
> I think we should consider these *.inc files similar to header files in
> c, right? For c libraries, you often see *-dev packages. If you think
> the dependency for source packages is to heavy, should we start to make
> such -dev packages as well for the pascal units? I think doublecmd also
> depends on fpc-source to include such an *.inc file (although that could
> be patched out and the same information could be obtained otherwise).
> There it works, unfortunately in the case of cge-src the file is not
> located on the fpc search path.
Generally pascal coding style does not encourage this kind of practice.
Include files should rather be used within the same sources in order to
provide different implementations for target specific code or to write
multiple units sharing the same code, but not meant to be distributed
like C headers. I don't think -dev packages are worth the effort
especially that we are dealing with 1 file and that is probably a
programming error not a real need.

> Until we create a better solution, I believe for view3dscene we should
> depend on the cge-src package. I will propose a solution as alternative
> to the patch in this weekend.
I'd personally prefer that we keep the patch as to say to upstream you
need to clean your code and remove this. But if you uploaded with the
dependency it is fine for me. However in this case I'd suggest to fill a
bug against view3dscene as it is not acceptable to depend on any -src
packages.

Cheers,
Abou Al Montacir
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 230 bytes
Desc: This is a digitally signed message part
URL: <http://lists.alioth.debian.org/pipermail/pkg-pascal-devel/attachments/20140610/3e4eabc6/attachment.sig>