kicad-developers team mailing list archive
-
kicad-developers team
-
Mailing list archive
-
Message #29573
Re: Patch: Open Datasheet in Project Dir
Does it not work by using the KIPRJMOD as is?
2017-05-31 15:17 GMT+02:00 Cheng Sheng <jeru.sheng@xxxxxxxxx>:
> Thanks, Fabrizio. It's good to know such info that is not mentioned in the
> doc. I'm indeed quite new to contributing code here.
>
> Regards,
> Cheng
>
> On 31 May 2017 at 13:06, Fabrizio Tappero <fabrizio.tappero@xxxxxxxxx>
> wrote:
>>
>> Hi Cheng,
>>
>> Don't worry you are doing just fine.
>>
>> Whenever somebody submits a patch, he does exactly what you did. Normally
>> it takes some days for the patch to get submitted because in these days
>> several people express opinions and feedback, as in your case.
>>
>> If nobody is against it and if the patch is useful for KiCad, the patch is
>> applied to the main repo in few days. A gentle reminder is a good way to
>> communicate with people with kicad repo writing rights.
>>
>> I hope you will submit more stuff. Contributing can be quite a rewarding
>> experience, I think. Just remember that the dev. community decides what goes
>> in and what stays out. And normally common good is paramount.
>>
>> cheers
>> Fabrizio
>>
>>
>>
>>
>>
>>
>>
>> On Wed, May 31, 2017 at 11:49 AM, Cheng Sheng <jeru.sheng@xxxxxxxxx>
>> wrote:
>>>
>>> okay, thanks for the info, Sergey.
>>>
>>> Don't find any explicit rule on how to select reviewers on the how to
>>> contribute page. So cc Jean-Pierre because you did a lot of commits
>>> recently. Let me know or reassign if there's some duty partitioning. Thanks.
>>>
>>> Regards,
>>> Cheng
>>>
>>> On 31 May 2017 at 11:01, Sergey A. Borshch <sb-sf@xxxxxxxxxxxxxxxxxxxxx>
>>> wrote:
>>>>
>>>> On 31.05.2017 10:57, Cheng Sheng wrote:
>>>>>
>>>>> So is there any conclusion on this patch like it will be accepted or
>>>>> rejected or needs some improvement?
>>>>>
>>>>> Sergey, are you a patch reviewer who can commit to the repo, or I need
>>>>> to add someone explicitly? Thanks.
>>>>
>>>> No, I'm just a user who wanted to share it's point of view.
>>>>
>>>>> Regards,
>>>>> Cheng
>>>>>
>>>>> On May 29, 2017 12:33 PM, "Cheng Sheng" <jeru.sheng@xxxxxxxxx
>>>>> <mailto:jeru.sheng@xxxxxxxxx>> wrote:
>>>>>
>>>>>
>>>>>
>>>>> On 29 May 2017 at 09:40, Sergey A. Borshch
>>>>> <sb-sf@xxxxxxxxxxxxxxxxxxxxx
>>>>> <mailto:sb-sf@xxxxxxxxxxxxxxxxxxxxx>> wrote:
>>>>>
>>>>> On 28.05.2017 23:49, Cheng Sheng wrote:
>>>>>
>>>>> I myself prefer storing all files specific to the project
>>>>> within the
>>>>> project directory, including datasheets, components,
>>>>> adjusted
>>>>> footprints and scripts. This way makes things easier when I
>>>>> copy the
>>>>> project around. After all, I don't mind disk space waste on
>>>>> a few
>>>>> duplicated datasheets; they don't take much space.
>>>>>
>>>>> Ok, what is your workflow? You choose component, pick it from
>>>>> library
>>>>> into schematics and then manually copy it's datasheet (from
>>>>> where?) into
>>>>> the project folder to make it available from schematics?
>>>>>
>>>>> I'm not a professional on electronics so I don't (want/bother to)
>>>>> maintain a
>>>>> centralized collection of datasheets. Instead, when I select
>>>>> components, I
>>>>> do parametric search on digikey/mouser/whatever, then check
>>>>> datasheets on
>>>>> the website, download to project dir if chosen.
>>>>>
>>>>>
>>>>> I have my library and my project both under version control
>>>>> system, so I
>>>>> use latest, exactly the same (with all latest error fixes)
>>>>> library and
>>>>> latest, most actual datasheets in every project. But I always
>>>>> can easily
>>>>> revert library repository to state it was when project was
>>>>> finished.
>>>>>
>>>>> I admire your patience and professionalism. I see your spirit to
>>>>> reuse as
>>>>> many things as possible. Shared library has its reuse benefit. Of
>>>>> course,
>>>>> also has its downsides. Eg., I don't necessarily remember how the
>>>>> shared
>>>>> libraries are setup one year after I finish a project or after I
>>>>> move to a
>>>>> new machine. So I prefer each project to be as self-containing as
>>>>> possible.
>>>>>
>>>>> Of course, putting everything inside the project also has its
>>>>> downsides,
>>>>> maybe even more than your proposed shared library approach. But I
>>>>> suppose
>>>>> this is just personal preferences. Unless there's a strong reason
>>>>> why my
>>>>> personal preference is terrible (that it is worse than a "standard
>>>>> approach"
>>>>> in ALL aspects) or by following this preference my patch makes the
>>>>> software
>>>>> terrible, it shouldn't be forbidden, right?
>>>>>
>>>>>
>>>>> There is "Archive library" option in eeschema File menu, which
>>>>> extract
>>>>> from library all components used in project. It can be extended
>>>>> to
>>>>> extract datasheets as well if it doesn't do it right now.
>>>>>
>>>>> hm... Am I missing some compile options? I don't see such an option
>>>>> in
>>>>> eevschema. Instead, only an "Archive Current Project" item in the
>>>>> project
>>>>> window.
>>>>>
>>>>> Anyway, even if datasheet extraction is added to "Archive library",
>>>>> you'll
>>>>> still need a way to open a relative datasheet unless the archiving
>>>>> leaves an
>>>>> absolute path (which seems unlikely the case, by guessing).
>>>>>
>>>>>
>>>>>
>>>>> If the direction of move by the patch makes some other
>>>>> workflow
>>>>> styles harder to optimize in the future, please let me
>>>>> know. I don't
>>>>> mind alternative proposals that can fulfill more workflow
>>>>> types.
>>>>> What I wrote is just a "shortest-path" I can see based on
>>>>> my own
>>>>> workflow.
>>>>>
>>>>> Regards,
>>>>> Cheng
>>>>>
>>>>> On 28 May 2017 at 20:53, Ingo Kletti
>>>>> <ikletti@xxxxxxxxxxxxxxxx
>>>>> <mailto:ikletti@xxxxxxxxxxxxxxxx>
>>>>> <mailto:ikletti@xxxxxxxxxxxxxxxx
>>>>>
>>>>> <mailto:ikletti@xxxxxxxxxxxxxxxx>>> wrote:
>>>>>
>>>>> Am 28.05.2017 um 18:47 schrieb Sergey A. Borshch:
>>>>>
>>>>> On 28.05.2017 14:35, Cheng Sheng wrote:
>>>>>
>>>>> So I made a patch to resolve the path before
>>>>> it is
>>>>> passed to
>>>>> "wxLaunchDefaultBrowser()". If it looks like a
>>>>> URL or
>>>>> is an absolute
>>>>> path, doesn't do anything; but if it is a
>>>>> relative
>>>>> path, append
>>>>> "${KIPRJMOD}" to it.
>>>>>
>>>>> Are you storing all datasheets in every project? I
>>>>> think
>>>>> it's better to
>>>>> keep one copy per library, so path must be
>>>>> relative to
>>>>> library location.
>>>>>
>>>>>
>>>>> Don't know about the OP, but from past experience it
>>>>> is wise to
>>>>> store the
>>>>> datasheet for the different components tpgether with
>>>>> the
>>>>> project files. That
>>>>> way you always have the specs available that your
>>>>> design is
>>>>> based on.
>>>>>
>>>>> So a relative path makes sense depending on the use
>>>>> case and
>>>>> workflow.
>>>>>
>>>>> Regards,
>>>>>
>>>>> Ingo
>>>>>
>>>>> --
>>>>> Regards,
>>>>> Sergey A. Borshch mailto: sb-sf@xxxxxxxxxxxxxxx
>>>>> <mailto:sb-sf@xxxxxxxxxxxxxxx>
>>>>> SB ELDI ltd. Riga, Latvia
>>>>>
>>>>> _______________________________________________
>>>>> Mailing list: https://launchpad.net/~kicad-developers
>>>>> <https://launchpad.net/~kicad-developers>
>>>>> Post to : kicad-developers@xxxxxxxxxxxxxxxxxxx
>>>>> <mailto:kicad-developers@xxxxxxxxxxxxxxxxxxx>
>>>>> Unsubscribe : https://launchpad.net/~kicad-developers
>>>>> <https://launchpad.net/~kicad-developers>
>>>>> More help : https://help.launchpad.net/ListHelp
>>>>> <https://help.launchpad.net/ListHelp>
>>>>>
>>>>>
>>>>>
>>>>
>>>>
>>>> --
>>>> Regards,
>>>> Sergey A. Borshch mailto: sb-sf@xxxxxxxxxxxxxxx
>>>> SB ELDI ltd. Riga, Latvia
>>>
>>>
>>>
>>> _______________________________________________
>>> 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
>
Follow ups
References
-
Patch: Open Datasheet in Project Dir
From: Cheng Sheng, 2017-05-28
-
Re: Patch: Open Datasheet in Project Dir
From: Sergey A. Borshch, 2017-05-28
-
Re: Patch: Open Datasheet in Project Dir
From: Ingo Kletti, 2017-05-28
-
Re: Patch: Open Datasheet in Project Dir
From: Cheng Sheng, 2017-05-28
-
Re: Patch: Open Datasheet in Project Dir
From: Sergey A. Borshch, 2017-05-29
-
Re: Patch: Open Datasheet in Project Dir
From: Cheng Sheng, 2017-05-29
-
Re: Patch: Open Datasheet in Project Dir
From: Cheng Sheng, 2017-05-31
-
Re: Patch: Open Datasheet in Project Dir
From: Sergey A. Borshch, 2017-05-31
-
Re: Patch: Open Datasheet in Project Dir
From: Cheng Sheng, 2017-05-31
-
Re: Patch: Open Datasheet in Project Dir
From: Fabrizio Tappero, 2017-05-31
-
Re: Patch: Open Datasheet in Project Dir
From: Cheng Sheng, 2017-05-31