kicad-developers team mailing list archive
-
kicad-developers team
-
Mailing list archive
-
Message #29574
Re: Patch: Open Datasheet in Project Dir
Hi Nick,
Could you be a little more specific? Do you mean:
- "KIPRJMOD/Datasheets/tmp.pdf"? or
- "${KIPRJMOD}/Datasheets/tmp.pdf", or
- "/path/to/my/project/dir/Datasheets/tmp.pdf" when
KIPRJMOD=/path/to/my/project/dir?
The first two doesn't work because the strings are passed to the browser
address as is. The third case is not good because if I move the project
around, the path becomes invalid.
Or none of the cases above is your proposal?
Thanks.
Regards,
Cheng
On 31 May 2017 at 17:23, Nick Østergaard <oe.nick@xxxxxxxxx> wrote:
> 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
-
Re: Patch: Open Datasheet in Project Dir
From: Nick Østergaard, 2017-05-31