kicad-developers team mailing list archive

[Wayne Stambaugh <stambaughw@xxxxxxxxx>]

I recant my original position that it would not be a trivial fix.  I
just pushed the fix to the master branch.  Please test it and let me
know if you find any issues.  I will be out of town over the weekend so
I may not reply as quickly as I normally do.

Please note that this fix isn't ideal.  Since Pcbnew does not support
odd numbered copper layers, selecting front or back layer only
configuration in the layer select dialog will not remove objects from
the copper layers.  It will remove them from all other layers if they
are not part of a footprint.  Even though this isn't technically
correct, it will (should?) prevent any false positive DRC tests from
occurring because objects will only exist on layers that exist.  Keep in
mind that this operation cannot be undone so once you save the board
after removing layers, that information is lost.  If you are not sure
about the change, don't save the board.

I will now resume my previously scheduled symbol library table work.

Cheers,

Wayne

On 9/28/2017 3:18 AM, Kristoffer Ödmark wrote:
> I agree as well, I've been hit by this bug once. And it was enough. I
> was actually planning to try and fix it similar to the way you are
> proposing right after the copypasta thingy is done.
> 
> ( My ideal solution would be to add a "remove items on extra layers"
> button to the dialog that would become enabled when removing layers, and
> simultaneously disbling the OK button, and only by pressing that would
> the OK button become enabled. )
> 
> On 09/28/2017 01:32 AM, David Godfrey wrote:
>> Hi Wayne,
>>
>> I'll back you on this decision.
>> It's more than once cost both time and money, to say nothing of
>> frustration to multiple users.
>>
>> Personally, I've not been caught with sending a board to production in
>> this state, but I certainly have run into the problem and caught it
>> during routing (Obviously non connected pins that don't DRC)
>>
>> If a layer is deleted, it's deleted, there simply shouldn't be any
>> leftovers of any description.
>> A clear warning that this is happening and confirmation dialog is
>> really all that is needed, especially if that dialog explicitly lists
>> the layers that are being removed.
>>
>>
>> On 27/09/17 22:08, Wayne Stambaugh wrote:
>>> This bug[1] has reared it's ugly head again so I am going to fix it once
>>> and for all before the stable 5 release.  We cannot continue storing
>>> objects on deleted layers in the board file which cause the DRC to pass
>>> connection tests when in reality there are failures.  I don't really
>>> have the time so you should assume that this is going push back the
>>> release.  It probably should be backported to the stable release but
>>> given the code deviations, I don't see that happening unless someone
>>> else feels motivated to do so.  I looked at the existing code and it's
>>> not going to be a trivial fix.  I do not intend to do anything fancy
>>> such as give the user the option to merge objects from deleted layers to
>>> other layers.  I'm going to keep it simple by just warning the user and
>>> removing all objects from the removed layers.  This includes non-copper
>>> layers as well.  I don't see how keeping objects on layers that don't
>>> exist in the board is valid.  I'm not really asking for comments but if
>>> you have any suggestions outside what I have presented I am willing to
>>> entertain them.
>>>
>>> [1]: https://bugs.launchpad.net/kicad/+bug/893950
>>>
>>> _______________________________________________
>>> Mailing list: https://launchpad.net/~kicad-developers
>>> Post to     : kicad-developers@xxxxxxxxxxxxxxxxxxx
>>> Unsubscribe : https://launchpad.net/~kicad-developers
>>> More help   : https://help.launchpad.net/ListHelp
>>>
>> Regards
>> David Godfrey
>> SB Tech Services
>> mb: +61 437 286 200 <tel:+61437286200>
>>
>>
>> _______________________________________________
>> Mailing list: https://launchpad.net/~kicad-developers
>> Post to     : kicad-developers@xxxxxxxxxxxxxxxxxxx
>> Unsubscribe : https://launchpad.net/~kicad-developers
>> More help   : https://help.launchpad.net/ListHelp
> 
> _______________________________________________
> Mailing list: https://launchpad.net/~kicad-developers
> Post to     : kicad-developers@xxxxxxxxxxxxxxxxxxx
> Unsubscribe : https://launchpad.net/~kicad-developers
> More help   : https://help.launchpad.net/ListHelp