DICOM @ OFFIS

Discussion Forum for OFFIS DICOM Tools - For registration, send email with desired user name to the OFFIS DICOM team
It is currently Wed, 2018-11-21, 11:51

All times are UTC + 1 hour




Post new topic Reply to topic  [ 48 posts ]  Go to page Previous  1, 2, 3, 4
Author Message
PostPosted: Thu, 2018-11-08, 13:34 
Offline

Joined: Thu, 2018-10-11, 11:10
Posts: 30
Sir

this #undef pw_gecos fix worked for me.
26 .so files are created.
Just need to know if this will have an impact on later stage and secondly if you are going to update dcmtk according to this change?


Top
 Profile  
 
PostPosted: Thu, 2018-11-08, 14:43 
Offline
OFFIS DICOM Team
OFFIS DICOM Team

Joined: Mon, 2014-03-03, 09:51
Posts: 282
Location: Oldenburg, Germany
ashajg wrote:
Just need to know if this will have an impact on later stage
Probably not, but one can't be sure. You will run into problems if you use code that depends on the define and also includes (directly or indirectly) ofstd/ofpwd.h. My guess would be that this is very unlikely, but you cannot be 100% sure.

ashajg wrote:
If you are going to update dcmtk according to this change?

I think I will add some code as I posted before that #undefines pw_gecos inside ofpwd.h and defines it back to pw_passwd afterwards as this should be the most robust solution.


Top
 Profile  
 
PostPosted: Fri, 2018-11-09, 11:58 
Offline

Joined: Thu, 2018-10-11, 11:10
Posts: 30
Jan Schlamelcher wrote:
ashajg wrote:
Just need to know if this will have an impact on later stage
Probably not, but one can't be sure. You will run into problems if you use code that depends on the define and also includes (directly or indirectly) ofstd/ofpwd.h. My guess would be that this is very unlikely, but you cannot be 100% sure.

ashajg wrote:
If you are going to update dcmtk according to this change?

I think I will add some code as I posted before that #undefines pw_gecos inside ofpwd.h and defines it back to pw_passwd afterwards as this should be the most robust solution.


OK sir

Thank you very much for help :D


Top
 Profile  
 
Display posts from previous:  Sort by  
Post new topic Reply to topic  [ 48 posts ]  Go to page Previous  1, 2, 3, 4

All times are UTC + 1 hour


Who is online

Users browsing this forum: No registered users and 1 guest


You cannot post new topics in this forum
You cannot reply to topics in this forum
You cannot edit your posts in this forum
You cannot delete your posts in this forum

Search for:
Jump to:  
cron
Powered by phpBB® Forum Software © phpBB Group