Opened 3 years ago

Closed 3 years ago

Last modified 3 years ago

#520 closed defect (fixed)

lasinfo reports missing msvcp120.dll

Reported by: martinl Owned by: osgeo4w-dev@…
Priority: major Component: Package
Version: Keywords: las, msvcrt
Cc:

Description

After installing las package, the command utilities do not work because of missing msvcp120.dll library. The problem seems to be that msvcrt package depends only on msvcrt2008 and msvcrt2010 packages (1). So in this case dependency on mscvrt2013 is missing in setup.hint. Is there any reason why this dependency (and 2012 version) is missing or is it just a bug?

(1) http://download.osgeo.org/osgeo4w/x86_64/release/msvcrt/setup.hint

Change History (10)

comment:1 Changed 3 years ago by martinl

I can fix the issue, I am just not sure what option is better:

1) add msvcrt2013 depenency to msvcrt package

2) add msvcrt2013 dependecy to las package

?

comment:2 Changed 3 years ago by jef

the package should require the individual msvcrt* package it requires - so that old msvcrt* package can be eventually removed, once no packages depend on it anymore.

Last edited 3 years ago by jef (previous) (diff)

comment:3 in reply to:  2 ; Changed 3 years ago by martinl

Replying to jef:

the package should require the individual msvcrt* package it requires - so that old msvcrt* package can be eventually removed, once no packages depend on it anymore.

so you are suggesting to add msvcrt2012 and msvcrt2013 as dependencies for msvcrt package, right?

comment:4 in reply to:  3 ; Changed 3 years ago by jef

Replying to martinl:

so you are suggesting to add msvcrt2012 and msvcrt2013 as dependencies for msvcrt package, right?

No. If packages depend on msvcrt instead of the individual msvcrtXXXX package, we'll neither know on which it actually depends and in turn will never be able to remove any of them - even if we actually don't need them anymore.

Last edited 3 years ago by jef (previous) (diff)

comment:5 in reply to:  4 Changed 3 years ago by martinl

Replying to jef:

Replying to martinl:

so you are suggesting to add msvcrt2012 and msvcrt2013 as dependencies for msvcrt package, right?

No. If packages depend on msvcrt instead of the individual msvcrtXXXX package, we'll neither know on which it actually depends and in turn will never be able to remove any of them - even if we actually don't need them anymore.

ah, sorry I was ready to quickly.

comment:6 in reply to:  4 Changed 3 years ago by martinl

Replying to jef:

Replying to martinl:

so you are suggesting to add msvcrt2012 and msvcrt2013 as dependencies for msvcrt package, right?

No. If packages depend on msvcrt instead of the individual msvcrtXXXX package, we'll neither know on which it actually depends and in turn will never be able to remove any of them - even if we actually don't need them anymore.

I have updated http://download.osgeo.org/osgeo4w/x86_64/release/liblas/setup.hint to include also msvcrt2013 package as a new dependency. I am not sure about 32bit version (which is 1.7.0 anyway).

comment:7 Changed 3 years ago by martinl

HUH, how I can close the ticket? In "Modify Ticket" section I can just see "Leave as new" and nothing else. Anyone else experiencing the same problem?

comment:8 Changed 3 years ago by jef

You should now be able to modify tickets.

comment:9 Changed 3 years ago by martinl

Resolution: fixed
Status: newclosed

comment:10 in reply to:  8 Changed 3 years ago by martinl

Replying to jef:

You should now be able to modify tickets.

Works, thanks.

Note: See TracTickets for help on using tickets.