Changeset 806d761 in mainline for uspace/lib/tbarcfg


Ignore:
Timestamp:
2024-02-07T23:44:59Z (2 years ago)
Author:
Jiri Svoboda <jiri@…>
Branches:
master
Children:
242e3c3
Parents:
74cb6610
Message:

Start menu should have 'open in terminal' functionality

Makes it easier for the user and if we are running in console mode,
we correctly start the application, instead of failing to start a
terminal.

Location:
uspace/lib/tbarcfg
Files:
4 edited

Legend:

Unmodified
Added
Removed

  • uspace/lib/tbarcfg/include/tbarcfg/tbarcfg.h

    r74cb6610 r806d761  
    11/*
    2  * Copyright (c) 2023 Jiri Svoboda
     2 * Copyright (c) 2024 Jiri Svoboda
    33 * All rights reserved.
    44 *
    … …  
    3939#include <errno.h>
    4040#include <sif.h>
     41#include <stdbool.h>
    4142#include <types/tbarcfg/tbarcfg.h>
    4243
    … …  
    4849extern const char *smenu_entry_get_caption(smenu_entry_t *);
    4950extern const char *smenu_entry_get_cmd(smenu_entry_t *);
     51extern bool smenu_entry_get_terminal(smenu_entry_t *);
    5052extern errno_t smenu_entry_set_caption(smenu_entry_t *, const char *);
    5153extern errno_t smenu_entry_set_cmd(smenu_entry_t *, const char *);
     54extern void smenu_entry_set_terminal(smenu_entry_t *, bool);
    5255extern errno_t smenu_entry_save(smenu_entry_t *);
    5356extern errno_t smenu_entry_create(tbarcfg_t *, const char *, const char *,
    54     smenu_entry_t **);
     57    bool, smenu_entry_t **);
    5558extern errno_t smenu_entry_destroy(smenu_entry_t *);
    5659

  • uspace/lib/tbarcfg/private/tbarcfg.h

    r74cb6610 r806d761  
    11/*
    2  * Copyright (c) 2023 Jiri Svoboda
     2 * Copyright (c) 2024 Jiri Svoboda
    33 * All rights reserved.
    44 *
    … …  
    4040#include <adt/list.h>
    4141#include <sif.h>
     42#include <stdbool.h>
    4243#include <types/tbarcfg/tbarcfg.h>
    4344
    … …  
    6465        /** Command to run */
    6566        char *cmd;
     67        /** Start in terminal */
     68        bool terminal;
    6669};
    6770
    6871extern errno_t smenu_entry_new(tbarcfg_t *, sif_node_t *, const char *,
    69     const char *, smenu_entry_t **);
     72    const char *, bool, smenu_entry_t **);
    7073extern void smenu_entry_delete(smenu_entry_t *);
    7174

  • uspace/lib/tbarcfg/src/tbarcfg.c

    r74cb6610 r806d761  
    11/*
    2  * Copyright (c) 2023 Jiri Svoboda
     2 * Copyright (c) 2024 Jiri Svoboda
    33 * All rights reserved.
    44 *
    … …  
    101101 * @return EOK on success or an error code
    102102 */
     103#include <stdio.h>
    103104errno_t tbarcfg_open(const char *repopath, tbarcfg_t **rtbcfg)
    104105{
    … …  
    110111        const char *caption;
    111112        const char *cmd;
     113        const char *terminal = NULL;
    112114        errno_t rc;
    113115
    … …  
    154156                }
    155157
    156                 rc = smenu_entry_new(tbcfg, nentry, caption, cmd, NULL);
     158                terminal = sif_node_get_attr(nentry, "terminal");
     159                if (terminal == NULL)
     160                        terminal = "n";
     161
     162                printf("terminal=%s\n", terminal);
     163
     164                rc = smenu_entry_new(tbcfg, nentry, caption, cmd,
     165                    str_cmp(terminal, "y") == 0, NULL);
    157166                if (rc != EOK)
    158167                        goto error;
    … …  
    233242/** Get start menu entry command.
    234243 *
    235  * @param entr Start menu entry
     244 * @param entry Start menu entry
    236245 * @return Command to run
    237246 */
    … …  
    239248{
    240249        return entry->cmd;
     250}
     251
     252/** Get start menu start in terminal flag.
     253 *
     254 * @param entry Start menu entry
     255 * @return Start in terminal flag
     256 */
     257bool smenu_entry_get_terminal(smenu_entry_t *entry)
     258{
     259        return entry->terminal;
    241260}
    242261
    … …  
    285304}
    286305
     306/** Set start menu entry start in terminal flag.
     307 *
     308 * Note: To make the change visible to others and persistent,
     309 * you must call @c smenu_entry_save()
     310 *
     311 * @param entry Start menu entry
     312 * @param terminal Start in terminal flag
     313 */
     314void smenu_entry_set_terminal(smenu_entry_t *entry, bool terminal)
     315{
     316        entry->terminal = terminal;
     317}
     318
    287319/** Save any changes to start menu entry.
    288320 *
    … …  
    303335
    304336        rc = sif_node_set_attr(trans, entry->nentry, "caption", entry->caption);
     337        if (rc != EOK)
     338                goto error;
     339
     340        rc = sif_node_set_attr(trans, entry->nentry, "terminal",
     341            entry->terminal ? "y" : "n");
    305342        if (rc != EOK)
    306343                goto error;
    … …  
    325362 * @param caption Caption
    326363 * @param cmd Command to run
     364 * @param terminal Start in terminal
    327365 * @param rentry Place to store pointer to new entry or @c NULL
    328366 */
    329367errno_t smenu_entry_new(tbarcfg_t *smenu, sif_node_t *nentry,
    330     const char *caption, const char *cmd, smenu_entry_t **rentry)
     368    const char *caption, const char *cmd, bool terminal, smenu_entry_t **rentry)
    331369{
    332370        smenu_entry_t *entry;
    … …  
    352390                goto error;
    353391        }
     392
     393        entry->terminal = terminal;
    354394
    355395        entry->smenu = smenu;
    … …  
    391431 * @param caption Caption
    392432 * @param cmd Command to run
     433 * @param terminal Start in terminal
    393434 * @param rentry Place to store pointer to new entry or @c NULL
    394435 */
    395436errno_t smenu_entry_create(tbarcfg_t *smenu, const char *caption,
    396     const char *cmd, smenu_entry_t **rentry)
     437    const char *cmd, bool terminal, smenu_entry_t **rentry)
    397438{
    398439        sif_node_t *nentry;
    … …  
    418459                goto error;
    419460
    420         rc = smenu_entry_new(smenu, nentry, caption, cmd, &entry);
     461        rc = sif_node_set_attr(trans, nentry, "terminal", terminal ? "y" : "n");
     462        if (rc != EOK)
     463                goto error;
     464
     465        rc = smenu_entry_new(smenu, nentry, caption, cmd, terminal ? "y" : "n",
     466            &entry);
    421467        if (rc != EOK)
    422468                goto error;

  • uspace/lib/tbarcfg/test/tbarcfg.c

    r74cb6610 r806d761  
    11/*
    2  * Copyright (c) 2023 Jiri Svoboda
     2 * Copyright (c) 2024 Jiri Svoboda
    33 * All rights reserved.
    44 *
    … …  
    7676        PCUT_ASSERT_ERRNO_VAL(EOK, rc);
    7777
    78         rc = smenu_entry_create(tbcfg, "A", "a", &e1);
     78        rc = smenu_entry_create(tbcfg, "A", "a", false, &e1);
    7979        PCUT_ASSERT_ERRNO_VAL(EOK, rc);
    8080        PCUT_ASSERT_NOT_NULL(e1);
    8181
    82         rc = smenu_entry_create(tbcfg, "B", "b", &e2);
     82        rc = smenu_entry_create(tbcfg, "B", "b", false, &e2);
    8383        PCUT_ASSERT_ERRNO_VAL(EOK, rc);
    8484        PCUT_ASSERT_NOT_NULL(e2);
    8585
    8686        /* Create entry without getting a pointer to it */
    87         rc = smenu_entry_create(tbcfg, "C", "c", NULL);
     87        rc = smenu_entry_create(tbcfg, "C", "c", false, NULL);
    8888        PCUT_ASSERT_ERRNO_VAL(EOK, rc);
    8989
    … …  
    102102
    103103/** Getting menu entry properties */
    104 PCUT_TEST(get_caption_cmd)
     104PCUT_TEST(get_caption_cmd_term)
    105105{
    106106        errno_t rc;
    … …  
    110110        const char *caption;
    111111        const char *cmd;
    112 
    113         p = tmpnam(fname);
    114         PCUT_ASSERT_NOT_NULL(p);
    115 
    116         rc = tbarcfg_create(fname, &tbcfg);
    117         PCUT_ASSERT_ERRNO_VAL(EOK, rc);
    118 
    119         rc = smenu_entry_create(tbcfg, "A", "a", &e);
     112        bool terminal;
     113
     114        p = tmpnam(fname);
     115        PCUT_ASSERT_NOT_NULL(p);
     116
     117        rc = tbarcfg_create(fname, &tbcfg);
     118        PCUT_ASSERT_ERRNO_VAL(EOK, rc);
     119
     120        rc = smenu_entry_create(tbcfg, "A", "a", false, &e);
    120121        PCUT_ASSERT_ERRNO_VAL(EOK, rc);
    121122
    … …  
    124125        cmd = smenu_entry_get_cmd(e);
    125126        PCUT_ASSERT_STR_EQUALS("a", cmd);
     127        terminal = smenu_entry_get_terminal(e);
     128        PCUT_ASSERT_FALSE(terminal);
    126129
    127130        tbarcfg_close(tbcfg);
    … …  
    130133
    131134/** Setting menu entry properties */
    132 PCUT_TEST(set_caption_cmd)
     135PCUT_TEST(set_caption_cmd_term)
    133136{
    134137        errno_t rc;
    … …  
    138141        const char *caption;
    139142        const char *cmd;
    140 
    141         p = tmpnam(fname);
    142         PCUT_ASSERT_NOT_NULL(p);
    143 
    144         rc = tbarcfg_create(fname, &tbcfg);
    145         PCUT_ASSERT_ERRNO_VAL(EOK, rc);
    146 
    147         rc = smenu_entry_create(tbcfg, "A", "a", &e);
     143        bool terminal;
     144
     145        p = tmpnam(fname);
     146        PCUT_ASSERT_NOT_NULL(p);
     147
     148        rc = tbarcfg_create(fname, &tbcfg);
     149        PCUT_ASSERT_ERRNO_VAL(EOK, rc);
     150
     151        rc = smenu_entry_create(tbcfg, "A", "a", false, &e);
    148152        PCUT_ASSERT_ERRNO_VAL(EOK, rc);
    149153
    … …  
    152156        cmd = smenu_entry_get_cmd(e);
    153157        PCUT_ASSERT_STR_EQUALS("a", cmd);
     158        terminal = smenu_entry_get_terminal(e);
     159        PCUT_ASSERT_FALSE(terminal);
    154160
    155161        /* Set properties */
    … …  
    158164        rc = smenu_entry_set_cmd(e, "b");
    159165        PCUT_ASSERT_ERRNO_VAL(EOK, rc);
     166        smenu_entry_set_terminal(e, true);
    160167
    161168        rc = smenu_entry_save(e);
    … …  
    167174        cmd = smenu_entry_get_cmd(e);
    168175        PCUT_ASSERT_STR_EQUALS("b", cmd);
     176        terminal = smenu_entry_get_terminal(e);
     177        PCUT_ASSERT_TRUE(terminal);
    169178
    170179        tbarcfg_close(tbcfg);
    … …  
    197206        const char *caption;
    198207        const char *cmd;
    199 
    200         p = tmpnam(fname);
    201         PCUT_ASSERT_NOT_NULL(p);
    202 
    203         rc = tbarcfg_create(fname, &tbcfg);
    204         PCUT_ASSERT_ERRNO_VAL(EOK, rc);
    205 
    206         rc = smenu_entry_create(tbcfg, "A", "a", &e);
     208        bool terminal;
     209
     210        p = tmpnam(fname);
     211        PCUT_ASSERT_NOT_NULL(p);
     212
     213        rc = tbarcfg_create(fname, &tbcfg);
     214        PCUT_ASSERT_ERRNO_VAL(EOK, rc);
     215
     216        rc = smenu_entry_create(tbcfg, "A", "a", false, &e);
    207217        PCUT_ASSERT_ERRNO_VAL(EOK, rc);
    208218        PCUT_ASSERT_NOT_NULL(e);
    … …  
    212222        cmd = smenu_entry_get_cmd(e);
    213223        PCUT_ASSERT_STR_EQUALS("a", cmd);
     224        terminal = smenu_entry_get_terminal(e);
     225        PCUT_ASSERT_FALSE(terminal);
     226
     227        smenu_entry_destroy(e);
     228
     229        rc = smenu_entry_create(tbcfg, "B", "b", true, &e);
     230        PCUT_ASSERT_ERRNO_VAL(EOK, rc);
     231        PCUT_ASSERT_NOT_NULL(e);
     232
     233        caption = smenu_entry_get_caption(e);
     234        PCUT_ASSERT_STR_EQUALS("B", caption);
     235        cmd = smenu_entry_get_cmd(e);
     236        PCUT_ASSERT_STR_EQUALS("b", cmd);
     237        terminal = smenu_entry_get_terminal(e);
     238        PCUT_ASSERT_TRUE(terminal);
     239
     240        smenu_entry_destroy(e);
    214241
    215242        tbarcfg_close(tbcfg);
    … …  
    231258        PCUT_ASSERT_ERRNO_VAL(EOK, rc);
    232259
    233         rc = smenu_entry_create(tbcfg, "A", "a", &e);
     260        rc = smenu_entry_create(tbcfg, "A", "a", false, &e);
    234261        PCUT_ASSERT_ERRNO_VAL(EOK, rc);
    235262
Note: See TracChangeset for help on using the changeset viewer.