Skip to content
Snippets Groups Projects

Resolve "Support for migration to a new silecs version"

Merged m.nabywaniec requested to merge 58-silecs-migrate into master

Issue #58 (closed)

Edited by m.nabywaniec

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • al.schwinn
  • Tests run fine for me. I did quick code review .. as well mostly looks good !

    Will try to test an installed version as well .. reporting back.

    Maybe after we could merge !8 (merged)

    Yea, it's about time :slight_smile:

    Edited by al.schwinn
  • For testing a local install I installed silecs like that and set the environment accordingly:

    ./build_and_install.sh 2.3.0 /home/bel/schwinn/lnx/tmp/silecstest
    export SILECS_BASE=/home/bel/schwinn/lnx/tmp/silecstest/2.3.0/

    Than I copied the outdated XML from one of the migration test scripts into my existing SilecsTest123_DU and fixed the names.

    Running the migration on it gives me this error:

    [schwinn@asl754 ~]$ cd ~/tmp/silecstest/2.3.0/silecs-cli/
    [schwinn@asl754 silecs-cli]$ ./silecs -m /home/bel/schwinn/lnx/workspace-acc8/SilecsTest123_DU/src/SilecsTest123_DU.silecsdeploy 2.3.0
    Error during importing module: No module named 'genparam'
    using sileces base path: /home/bel/schwinn/lnx/tmp/silecstest/2.3.0/
    [Errno 20] Not a directory: '/home/bel/schwinn/lnx/workspace-acc8/SilecsTest123_DU/src/SilecsTest123_DU.silecsdeploy'

    ... so it seems like I need to pass a project-folder ? Though that as well did not work :slight_frown:

    Can you reproduce this bug ?

    Edited by al.schwinn
  • m.nabywaniec added 3 commits

    added 3 commits

    • 2bad7f44 - Fix finding silecs file during migration
    • 5d8edf5c - Remove redundant migration
    • 46adf602 - Remove src from paths

    Compare with previous version

  • m.nabywaniec added 1 commit

    added 1 commit

    • b4201392 - Fix importing modules for migration

    Compare with previous version

  • m.nabywaniec added 1 commit

    added 1 commit

    • 9bbaa319 - Remove migration module from modules to apply if not found

    Compare with previous version

  • m.nabywaniec added 1 commit

    added 1 commit

    • d9e8097e - Fix extracting module versions from migration modules

    Compare with previous version

  • m.nabywaniec added 1 commit

    added 1 commit

    • d443b6fc - Remove unused MigrationAny class

    Compare with previous version

  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Please register or sign in to reply
    Loading