Saturday, December 23, 2023

Re: Bugfix: lang/deno

I would prefer to clean it up right away.


Antoine

> On Dec 23, 2023, at 17:17, Volker Schlecht <openbsd-ports@schlecht.dev> wrote:
>
> On 2023-12-23 16:50, Antoine Jacoutot wrote:
>>> On Sat, Dec 23, 2023 at 04:46:40PM +0100, Volker Schlecht wrote:
>>> + fn op_exec_path(state: &mut OpState) -> Result<String, AnyError> {
>>> + let current_exe = env::current_exe().unwrap();
>>> ++ let current_exe =
>>> ++ std::env::current_exe().unwrap_or(PathBuf::from("${LOCALBASE}/bin/deno"));
>>
>> You probably want PREFIX or TRUEPREFIX here.
>
> In hindsight: Probably yes.
> However I'm already using LOCALBASE in 4 other patches for the same purpose
> in lang/deno. So for now, I would prefer consistency. I can clean this up with
> the next version upgrade (WIP).
>
> ok for now?
>

No comments:

Post a Comment