Introducing alid

[rikki cattermole]

On 13/09/2022 4:25 AM, Ali Çehreli wrote:
> On 9/12/22 07:43, rikki cattermole wrote:
>> Looks pretty well tested, nice!
> 
> Thanks! Proud with 100% coverage. :)

I was going to ask about coverage, that is awesome!

>> But in other less nice things, I take it you did not test with GDC? 
>> GDC does not support cli args with the same names as dmd. One of these 
>> is -mv.
> 
> So far, I started learning by copying arsd's dub.json. (Thank you, Adam! 
> The misunderstandings are mine. :) )

It has the same issues, which is why I recommended against copying it.

>> The file structure of subPackage/alid/subPackage will not require it 
>> and you will not have the cross import issues, where if you depend on 
>> errornogc you can also import (and then get linker errors) for 
>> circularblocks.
> 
> If I understand you correctly, the directory structure need to be the 
> following (also introducing src, which is clearly missing :)):
> 
> alid/src/errornogc/alid/errornogc.d
>       .../circularblocks/alid/circularblocks.d
> [...]

No, you don't need the src directory, the subPackage directory functions 
as this.

So:

dub.json
errornogc/alid/errornogc.d
circularblocks/alid/circularblocks.d

That'll work.

> Can I add a CI step to catch all such issues? It would be awesome if dub 
> provided that.

There is:

https://github.com/dlang-community/setup-dlang

But it doesn't look to support gdc or have been updated in a little 
while. Guess I need to start pinging people about it.


> Ali
> 
> P.S. Another issue is function attributes seemingly used inaccurately 
> but I asked that question on the 'learn' newsgroup already. Ping! ;)

I have no solution to that unfortunately beyond template all the things.