Skip to content

  • Projects
  • Groups
  • Snippets
  • Help
  • This project
    • Loading...
  • Sign in / Register
U
uniset2
  • Project
    • Project
    • Details
    • Activity
    • Cycle Analytics
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributors
    • Graph
    • Compare
    • Charts
  • Issues 1
    • Issues 1
    • List
    • Board
    • Labels
    • Milestones
  • Merge Requests 0
    • Merge Requests 0
  • CI / CD
    • CI / CD
    • Pipelines
    • Jobs
    • Schedules
    • Charts
  • Wiki
    • Wiki
  • Members
    • Members
  • Collapse sidebar
  • Activity
  • Graph
  • Charts
  • Create a new issue
  • Jobs
  • Commits
  • Issue Boards
  • UniSet project repositories
  • uniset2
  • Merge Requests
  • !1

Merged
Opened Sep 03, 2018 by Ghost User@ghost 
  • Report abuse
Report abuse

Добавил вывод на каком omni порте uniset2-start.sh будет запускать процессы

Не нашел как с помощью скрипта запуска посмотреть на каком порту он запустит исполняемый файл. Решил добавить. Мало ли не только мне приспичит.

Edited Sep 03, 2018 by Ghost User
  • Discussion 8
  • Commits 2
  • Changes 1
{{ resolvedDiscussionCount }}/{{ discussionCount }} {{ resolvedCountText }} resolved
  • Pavel Vainerman
    @pv started a discussion on an old version of the diff Sep 03, 2018
    Last updated by Ghost User Sep 03, 2018
    Utilities/scripts/uniset2-start.sh
    21 21
    22 22 Valid options are:
    23 23 -h, --help display help screen
    24 -o, --omni-port print defautl omni port number fot current user
    • Pavel Vainerman @pv commented · Sep 03, 2018
      Maintainer

      опечаточка fot ?

      Edited Sep 03, 2018 by Pavel Vainerman
      опечаточка `fot` ?
    • Pavel Vainerman @pv commented · Sep 03, 2018
      Maintainer

      defautl тоже

      `defautl` тоже
    • Ghost User @ghost

      changed this line in version 2 of the diff

      · Sep 03, 2018

      changed this line in version 2 of the diff

      changed this line in [version 2 of the diff](https://gitlab.eterfund.ru/pv/uniset2/merge_requests/1/diffs?diff_id=808&start_sha=649a732fc4f89c4865a54da3dc64c6c2fd68c878#5924b4a2e5fcec100cf93cd25724e722f6d86cc2_24_24)
      Toggle commit list
    Please register or sign in to reply
  • Pavel Vainerman
    @pv started a discussion on an old version of the diff Sep 03, 2018
    Last updated by Ghost User Sep 03, 2018
    Utilities/scripts/uniset2-start.sh
    48 50 esac
    49 51 shift
    50 52
    53 if [ -n "OPORT" ]
    54 then
    55 echo "Uniset default omni port for current user: $OMNIPORT"
    • Pavel Vainerman @pv commented · Sep 03, 2018
      Maintainer

      -o - это только вывод порта на экран и всё. Или просто вывод во время запуска? Я к тому, должен ли быть тут exit 0 поcле echo?

      -o - это только вывод порта на экран и всё. Или просто вывод во время запуска? Я к тому, должен ли быть тут `exit 0` поcле `echo`?
    • Pavel Vainerman @pv commented · Sep 03, 2018
      Maintainer

      А ещё я бы здесь выводил Uniset default omni port for '$USER': $OMNIPORT"

      А ещё я бы здесь выводил ```Uniset default omni port for '$USER': $OMNIPORT"```
    • Ghost User @ghost commented · Sep 03, 2018

      Только вывод, без запуска. Во время запуска там слишком много всего сыпется - толку будет немного.

      Только вывод, без запуска. Во время запуска там слишком много всего сыпется - толку будет немного.
    • Pavel Vainerman @pv commented · Sep 03, 2018
      Maintainer

      тогда должен быть exit 0

      тогда должен быть `exit 0`
    • Ghost User @ghost

      changed this line in version 2 of the diff

      · Sep 03, 2018

      changed this line in version 2 of the diff

      changed this line in [version 2 of the diff](https://gitlab.eterfund.ru/pv/uniset2/merge_requests/1/diffs?diff_id=808&start_sha=649a732fc4f89c4865a54da3dc64c6c2fd68c878#5924b4a2e5fcec100cf93cd25724e722f6d86cc2_55_55)
      Toggle commit list
    Please register or sign in to reply
  • Ghost User @ghost

    changed the description

    · Sep 03, 2018

    changed the description

    changed the description
    Toggle commit list
  • Ghost User @ghost

    assigned to @pv

    · Sep 03, 2018

    assigned to @pv

    assigned to @pv
    Toggle commit list
  • Ghost User @ghost commented · Sep 03, 2018

    Остальное поправлю

    Остальное поправлю
  • Pavel Vainerman
    @pv started a discussion on an old version of the diff Sep 03, 2018
    Last updated by Ghost User Sep 03, 2018
    Utilities/scripts/uniset2-start.sh
    40 41 case "$1" in
    41 42 -h|--help) print_usage 0;;
    42 43 -f|--foreground) FG=1;;
    44 -o|--omni-port) OPORT="omni-port";;
    • Pavel Vainerman @pv commented · Sep 03, 2018
      Maintainer

      А ещё мне кажется, здесь -f уехало, поэтому стоило всё-таки по отступу ориентироваться на других.. (и возможно подправить -f)

      Edited Sep 03, 2018 by Pavel Vainerman
      А ещё мне кажется, здесь `-f` уехало, поэтому стоило всё-таки по отступу ориентироваться на других.. (и возможно подправить -f)
    • Ghost User @ghost

      changed this line in version 2 of the diff

      · Sep 03, 2018

      changed this line in version 2 of the diff

      changed this line in [version 2 of the diff](https://gitlab.eterfund.ru/pv/uniset2/merge_requests/1/diffs?diff_id=808&start_sha=649a732fc4f89c4865a54da3dc64c6c2fd68c878#5924b4a2e5fcec100cf93cd25724e722f6d86cc2_44_43)
      Toggle commit list
    Please register or sign in to reply
  • Ghost User @ghost

    added 1 commit

    • f94fef08 - Подправил замечания по отступам, опечаткам и возвращению статуса процесса

    Compare with previous version

    · Sep 03, 2018

    added 1 commit

    • f94fef08 - Подправил замечания по отступам, опечаткам и возвращению статуса процесса

    Compare with previous version

    added 1 commit <ul><li>f94fef08 - Подправил замечания по отступам, опечаткам и возвращению статуса процесса</li></ul> [Compare with previous version](https://gitlab.eterfund.ru/pv/uniset2/merge_requests/1/diffs?diff_id=808&start_sha=649a732fc4f89c4865a54da3dc64c6c2fd68c878)
    Toggle commit list
  • Pavel Vainerman @pv

    merged

    · Sep 03, 2018

    merged

    merged
    Toggle commit list
  • Write
  • Preview
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or sign in to comment
Pavel Vainerman
Assignee
Pavel Vainerman @pv
Assign to
None
Milestone
None
Assign milestone
Time tracking
0
Labels
None
Assign labels
  • View project labels
Reference: uniset/uniset2!1