Project

General

Profile

Actions

Support Request #9404

closed

Conan ADTF and SDK packages do not add their bin directories to path

Added by hidden over 4 years ago. Updated about 4 years ago.

Status:
Closed
Priority:
Normal
Customer:
AUDI
Department:
EF
Requester's Priority:
Normal
Support Level:
3rd Level
Resolution:
Solved Issue
Product Issue Numbers:
Platform:
Windows 10 64bit
Topic:
ADTF::Common
FAQ Links:

Description

ADTF, the Toolboxes and especially the SDK packages like OSG shall add their bin directories to path in conanfile package_info:

def package_info(self):            
self.env_info.path.append(os.path.join(self.package_folder, "bin"))

Without this modules (e.g. filters) built against these SDK cannot be used with conan's native virtualenv generator.


Related issues

Related to Public Support - Support Request #11012: Using conan packages and required package _DIRsClosedActions
Related to Public Support - Support Request #10968: ADTF3 Conan Package Variants (EB vs. DW)ClosedActions
Actions #1

Updated by hidden over 4 years ago

  • Topic set to ADTF::Common
Actions #5

Updated by hidden over 4 years ago

  • Status changed from New to In Progress
  • Support Level changed from 2nd Level to 3rd Level

Nils ist bereits im Urlaub und hatte dazu Kontakt mit Falk.
Wird dann ab KW02/20 weiter bearbeitet.

Actions #6

Updated by hidden over 4 years ago

Ich habe das Due date auf KW 13 verschoben da z.Z. der nächste ADTF release Vorrang hat.

Schöne Grüße

Actions #7

Updated by hidden over 4 years ago

  • Status changed from In Progress to To Be Closed
  • Resolution set to Solved Issue

Ich bin aufgrund von Krankheit erst jetzt dazu gekommen.
Für OSG habe ich das conanfile angepasst. Auch wenn die Platform hier nicht angefordert war sage ich trotzdem direkt dazu: Unter Ubuntu funktioniert das nicht, da die libs nicht auf dem rpath sind. Falls Ubuntu gebraucht wird bitte ein neues Ticket aufmachen.

Wir stellen gerade unseren Deploy Prozess um in dem Zuge werde ich den path für virtualenv in den pkgs ergänzen in denen es Sinn ergibt.

Schöne Grüße

Actions #8

Updated by hidden over 4 years ago

  • Project changed from 11 to Public Support
  • Subject changed from [CONAN] ADTF and SDK packages do not add their bin directories to path to Conan ADTF and SDK packages do not add their bin directories to path
  • Private changed from Yes to No
Actions #9

Updated by hidden over 4 years ago

Actions #10

Updated by hidden about 4 years ago

Hallo,

wie heute besprochen würde ich gern noch einmal meine konkreten Wünsche bzg. conan-Rezepte der adtf/sdk-Pakete äußern. Ziel ist es, dass die Pakete über "native" conan nutzbar sind ohne cmake-magic aus den mitgelieferten ADTF-cmake-files oder n-faches DLL-Kopiere. Natürlich sollen die ADTF-cmake-scripte nicht ersetzt werden, lediglich die pakete so erweitert werden, dass auch libs und pfade innerhalb von conan korrekt gesetzt und gefunden werden. Mir ist bewusst, dass das u.U. unter unix nicht geht. Für uns ist unix aber aktuell kein Thema. Es bliebe auch immer der Weg über die mitgelieferten adtf-cmake-scripts offen. Ich sehe eigentlich keinen Grund, warum die package_info() nicht ergänzt werden kann. Das stört keinen, ist wenig Aufwand und würde massiv beim Entwickeln von neuen Plugins und v.a. Tooling mit und um Conan herum helfen (conan workspaces, jenkins jobs, packaging, adtf-startscript-generator, ...)

alle punkte gelten für ADTF, Toolboxen, Qt, OSG, ...:
- self.env_info.path - braucht man bei ADTF auch das /3rdparty/qt5/bin/?
- self.cpp_info.libs - im zweifelsfall einfach alle libs außer den 3rd und 3rd3rd party libs, solange das paket (ADTF) ein monolith ist.
- self.cpp_info.define - falls es da welche gibt (bei ADTF2 hatten wir z.b. ADTF_A_UTILS_LEGACY_NAMES gesetzt um nicht alle Filter anpassen zu müssen)
- self.cpp_info.bindirs (sofern nicht nur /bin/)
- self.cpp_info.libdirs (sofern nicht nur /lib/)
- self.cpp_info.includedirs (sofern nicht nur /include/)

leider hat es in die 3.7.0 nicht eine dieser Variablen geschafft :(

Actions #11

Updated by hidden about 4 years ago

  • Status changed from To Be Closed to In Progress

@Nils: Kannst du die Anforderungen nochmal festhalten, damit wir das künftig berücksichtigen (soweit möglich) ?
Ich denke Falk kann auch gut als Tester dienen, wenn wir ihm Beta Stände freischalten können (ggf. schon zur 3.8.0)
Deshalb bitte einfach mal im passenden Ticketsystem für die nächste Iteration verorten, danach kann dieses Supportticket abgeschlossen werden

Actions #12

Updated by hidden about 4 years ago

Florian Roth wrote:

@Nils: Kannst du die Anforderungen nochmal festhalten, damit wir das künftig berücksichtigen (soweit möglich) ?
Ich denke Falk kann auch gut als Tester dienen, wenn wir ihm Beta Stände freischalten können (ggf. schon zur 3.8.0)
Deshalb bitte einfach mal im passenden Ticketsystem für die nächste Iteration verorten, danach kann dieses Supportticket abgeschlossen werden

Tickets sind erstellt.

Actions #13

Updated by hidden about 4 years ago

  • Status changed from In Progress to To Be Closed
Actions #14

Updated by hidden about 4 years ago

Actions #15

Updated by hidden about 4 years ago

  • Status changed from To Be Closed to Closed
Actions

Also available in: Atom PDF